diff options
| author | Luca Deri <lucaderi@users.noreply.github.com> | 2020-04-21 19:23:57 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-04-21 19:23:57 +0200 |
| commit | 019b51bb178d83405d5c01ce604f240c0914f9df (patch) | |
| tree | 0ec2ba19365c66b9dba49cf35397762e5cbc030d /src/lib/protocols/tls.c | |
| parent | e2be7daed6b298e214df91f54061cdac91cd8781 (diff) | |
| parent | b1a6c6b8957b62ec8ada423abfdcfa5471c00147 (diff) | |
Merge pull request #879 from IvanNardi/warnings
Fix some compilation warnings
Diffstat (limited to 'src/lib/protocols/tls.c')
| -rw-r--r-- | src/lib/protocols/tls.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/lib/protocols/tls.c b/src/lib/protocols/tls.c index 560e483ac..a6d7883ce 100644 --- a/src/lib/protocols/tls.c +++ b/src/lib/protocols/tls.c @@ -316,8 +316,10 @@ static void processCertificateElements(struct ndpi_detection_module_struct *ndpi i += 2; - if((len > sizeof(dNSName)-1) || (len == 0 /* Looks something went wrong */)) - break; /* String too long */ + /* The check "len > sizeof(dNSName) - 1" will be always false. If we add it, + the compiler is smart enough to detect it and throws a warning */ + if((len == 0 /* Looks something went wrong */)) + break; strncpy(dNSName, (const char*)&packet->payload[i], len); dNSName[len] = '\0'; |