Project

General

Profile

Actions

Bug #903

closed

libhtp valgrind warning

Added by Victor Julien almost 11 years ago. Updated over 10 years ago.

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

Description

==11894== Thread 8:
==11894== Conditional jump or move depends on uninitialised value(s)
==11894==    at 0x60B7BA0: htp_tx_urldecode_uri_inplace (htp_util.c:1427)
==11894==    by 0x60B7E97: htp_normalize_parsed_uri (htp_util.c:1690)
==11894==    by 0x60B4C5C: htp_tx_state_request_line (htp_transaction.c:937)
==11894==    by 0x60B0D36: htp_connp_REQ_LINE (htp_request.c:706)
==11894==    by 0x60B0E78: htp_connp_req_data (htp_request.c:851)
==11894==    by 0x456968: HTPHandleRequestData (app-layer-htp.c:642)
==11894==    by 0x47F111: AppLayerDoParse (app-layer-parser.c:894)
==11894==    by 0x48061E: AppLayerParse (app-layer-parser.c:1100)
==11894==    by 0x41CFF8: AppLayerHandleTCPData (app-layer.c:161)
==11894==    by 0x8A9499: StreamTcpReassembleAppLayer (stream-tcp-reassemble.c:2933)
==11894==    by 0x8ABCE3: StreamTcpReassembleHandleSegmentUpdateACK (stream-tcp-reassemble.c:3295)
==11894==    by 0x8AC532: StreamTcpReassembleHandleSegment (stream-tcp-reassemble.c:3369)
==11894== 
==11894== Conditional jump or move depends on uninitialised value(s)
==11894==    at 0x60B7BB3: htp_tx_urldecode_uri_inplace (htp_util.c:1431)
==11894==    by 0x60B7E97: htp_normalize_parsed_uri (htp_util.c:1690)
==11894==    by 0x60B4C5C: htp_tx_state_request_line (htp_transaction.c:937)
==11894==    by 0x60B0D36: htp_connp_REQ_LINE (htp_request.c:706)
==11894==    by 0x60B0E78: htp_connp_req_data (htp_request.c:851)
==11894==    by 0x456968: HTPHandleRequestData (app-layer-htp.c:642)
==11894==    by 0x47F111: AppLayerDoParse (app-layer-parser.c:894)
==11894==    by 0x48061E: AppLayerParse (app-layer-parser.c:1100)
==11894==    by 0x41CFF8: AppLayerHandleTCPData (app-layer.c:161)
==11894==    by 0x8A9499: StreamTcpReassembleAppLayer (stream-tcp-reassemble.c:2933)
==11894==    by 0x8ABCE3: StreamTcpReassembleHandleSegmentUpdateACK (stream-tcp-reassemble.c:3295)
==11894==    by 0x8AC532: StreamTcpReassembleHandleSegment (stream-tcp-reassemble.c:3369)
==11894== 
==11894== Conditional jump or move depends on uninitialised value(s)
==11894==    at 0x60B7BC6: htp_tx_urldecode_uri_inplace (htp_util.c:1435)
==11894==    by 0x60B7E97: htp_normalize_parsed_uri (htp_util.c:1690)
==11894==    by 0x60B4C5C: htp_tx_state_request_line (htp_transaction.c:937)
==11894==    by 0x60B0D36: htp_connp_REQ_LINE (htp_request.c:706)
==11894==    by 0x60B0E78: htp_connp_req_data (htp_request.c:851)
==11894==    by 0x456968: HTPHandleRequestData (app-layer-htp.c:642)
==11894==    by 0x47F111: AppLayerDoParse (app-layer-parser.c:894)
==11894==    by 0x48061E: AppLayerParse (app-layer-parser.c:1100)
==11894==    by 0x41CFF8: AppLayerHandleTCPData (app-layer.c:161)
==11894==    by 0x8A9499: StreamTcpReassembleAppLayer (stream-tcp-reassemble.c:2933)
==11894==    by 0x8ABCE3: StreamTcpReassembleHandleSegmentUpdateACK (stream-tcp-reassemble.c:3295)
==11894==    by 0x8AC532: StreamTcpReassembleHandleSegment (stream-tcp-reassemble.c:3369)
==11894== 

This is from running Suri through valgrind on a very large pcap overnight, so extracting the flow is impossible.

Actions #1

Updated by Victor Julien almost 11 years ago

Trying to isolate the flow(s) using https://github.com/inliniac/suricata/pull/463

Actions #2

Updated by Ivan Ristic almost 11 years ago

  • Status changed from Assigned to Resolved

This was fixed a couple of days ago, in LibHTP v.0.5.6:

https://github.com/ironbee/libhtp/commit/2d2105a7225b266f56c46bf41945c32305c21177

Actions #3

Updated by Anoop Saldanha over 10 years ago

  • Status changed from Resolved to Closed
  • Assignee changed from Anoop Saldanha to Victor Julien
Actions

Also available in: Atom PDF