Project

General

Profile

Actions
Copy link

Bug #198

closed

valgrind report a context error on SSLParserTest01

Added by Pablo Rincon over 14 years ago. Updated over 14 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Pablo Rincon
Target version:
1.0.0
Affected Versions:
Effort:
Difficulty:
Label:

Description

valgrind sometimes reports the following error with -O2 on a 64bit vmware box:
pablo@kdevel:/home/pablo/oisf/src# file .libs/suricata
.libs/suricata: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked (uses shared libs), for GNU/Linux 2.6.15, not stripped

11613 Conditional jump or move depends on uninitialised value(s)
11613 at 0x5D6044: SSLParseClientRecord (app-layer-ssl.c:91)
11613 by 0x5B63EC: AppLayerDoParse (app-layer-parser.c:675)
11613 by 0x5B6F4D: AppLayerParse (app-layer-parser.c:879)
11613 by 0x5D7A10: SSLParserTest01 (app-layer-ssl.c:275)
11613 by 0x53A90A: UtRunTests (util-unittest.c:199)
11613 by 0x405E95: main (suricata.c:893)
11613
11613 ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 27 from 4)

Files

0002-This-patch-for-app-layer-ssl-fix-the-bug-198-SSLPa.patch (2.01 KB) 0002-This-patch-for-app-layer-ssl-fix-the-bug-198-SSLPa.patch It's really weird, but it seems that the way of initializing the input was not handled properly by gcc on -O2 and -O3. I changed it to avoid the usage of brackets as array, and directly use a string with hexa data, fixing the length, since it has \0's, an Pablo Rincon, 06/30/2010 10:12 AM
Actions
Copy link

Also available in: Atom PDF