Project

General

Profile

Actions
Copy link

Bug #978

closed

clean up app layer parser thread local storage

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

Status:
Closed
Priority:
Normal
Assignee:
Victor Julien
Target version:
2.0.1rc1
Affected Versions:
Effort:
Difficulty:
Label:

Description

Error #39: LEAK 20 direct bytes 0x08f1f950-0x08f1f964 + 95 indirect bytes
# 0 replace_malloc                                      [/work/drmemory_package/common/alloc_replace.c:2292]
# 1 SMTPLocalStorageAlloc                               [/home/victor/dev/oisf/src/app-layer-smtp.c:767]
# 2 AppLayerGetProtocolParserLocalStorage               [/home/victor/dev/oisf/src/app-layer-parser.c:776]
# 3 AlpProtoFinalizeThread                              [/home/victor/dev/oisf/src/app-layer-detect-proto.c:283]
# 4 AlpProtoFinalize2Thread                             [/home/victor/dev/oisf/src/app-layer-detect-proto.c:305]
# 5 StreamTcpReassembleInitThreadCtx                    [/home/victor/dev/oisf/src/stream-tcp-reassemble.c:372]
# 6 StreamTcpThreadInit                                 [/home/victor/dev/oisf/src/stream-tcp.c:4512]
# 7 TmThreadsSlotPktAcqLoop                             [/home/victor/dev/oisf/src/tm-threads.c:669]
# 8 libpthread.so.0!start_thread                        [/build/buildd/eglibc-2.17/nptl/pthread_create.c:311]
# 9 libc.so.6!__clone                                   [../sysdeps/unix/sysv/linux/i386/clone.S:131]
Actions
Copy link
 #1

Updated by Anoop Saldanha about 11 years ago

  • Assignee changed from OISF Dev to Anoop Saldanha

Already being carried out by my app layer changes.

Actions
Copy link
 #2

Updated by Victor Julien about 11 years ago

There is a DrMemory suppression for this, please remove it in the patchset that addresses this bug.

Actions
Copy link
 #3

Updated by Victor Julien about 11 years ago

  • Target version changed from 2.0beta2 to 2.0rc1
Actions
Copy link
 #4

Updated by Victor Julien about 11 years ago

  • Status changed from New to Assigned
  • Priority changed from Normal to High
Actions
Copy link
 #5

Updated by Victor Julien almost 11 years ago

  • Target version changed from 2.0rc1 to 2.0rc2
Actions
Copy link
 #6

Updated by Victor Julien almost 11 years ago

  • Assignee changed from Anoop Saldanha to Victor Julien
  • Priority changed from High to Normal
  • Target version changed from 2.0rc2 to 2.0.1rc1
  • % Done changed from 0 to 50

I have a fix, but it touches quite a bit of code. As it's a fairly minor issue, pushing it back to 2.0.1.

Actions
Copy link
 #7

Updated by Victor Julien over 10 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 50 to 100
Actions
Copy link

Also available in: Atom PDF