Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PVS-Studio fixes #34

Closed
rtsisyk opened this issue Jan 20, 2017 · 0 comments
Closed

PVS-Studio fixes #34

rtsisyk opened this issue Jan 20, 2017 · 0 comments
Labels
bug Something isn't working

Comments

@rtsisyk
Copy link
Contributor

rtsisyk commented Jan 20, 2017

memcached/internal/memcached.c:340 err V695 Range intersections are possible within conditional expressions. Example: if (A < 5) { ... } else if (A < 2) { ... }. Check lines: 338, 340.
memcached/internal/proto_bin.c:122 err V517 The use of 'if (A) {...} else if (A) {...}' pattern was detected. There is a probability of logical error presence. Check lines: 122, 126.
memcached/internal/proto_bin.c:534 err V547 Expression is always true. Probably the '&&' operator should be used here.
memcached/internal/proto_bin.c:1104 warn V547 Expression 'errstr' is always true.

@rtsisyk rtsisyk added the bug Something isn't working label Jan 20, 2017
@bigbes bigbes closed this as completed in a84d1f9 Jan 20, 2017
viciious pushed a commit to viciious/memcached that referenced this issue Feb 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant