diff options
author | Ilya Shipitsin <chipitsine@gmail.com> | 2016-04-29 22:22:05 +0300 |
---|---|---|
committer | Ilya Shipitsin <chipitsine@gmail.com> | 2016-04-29 22:22:05 +0300 |
commit | 29f93371c3d1917522dc4db074188d9ac2be30f8 (patch) | |
tree | 9774e90a19a9e1c69f51f77c5454d83804ac7c5f | |
parent | 01edd34bbe22f8500cb539102cbdbc143c197600 (diff) |
cppcheck issue:
[src/Cedar/Connection.c:1041] -> [src/Cedar/Connection.c:1043]: (warning) Either the condition 's!=0' is redundant or there is possible null pointer dereference: s.
-rw-r--r-- | src/Cedar/Connection.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/Cedar/Connection.c b/src/Cedar/Connection.c index ada6ecc2..517096e4 100644 --- a/src/Cedar/Connection.c +++ b/src/Cedar/Connection.c @@ -1030,7 +1030,7 @@ void ConnectionSend(CONNECTION *c, UINT64 now) UINT size;
SESSION *s;
HUB *hub = NULL;
- bool use_qos;
+ bool use_qos = false;
// Validate arguments
if (c == NULL)
{
@@ -1038,11 +1038,11 @@ void ConnectionSend(CONNECTION *c, UINT64 now) }
s = c->Session;
- use_qos = s->QoS;
if (s != NULL)
{
hub = s->Hub;
+ use_qos = s->QoS;
}
// Protocol
|