Project

General

Profile

Actions

Bug #170

closed

memory leak in FTP handling

Added by Victor Julien almost 14 years ago. Updated almost 14 years ago.

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

Description

It appears that the FTP app layer parser is leaking memory.

18927 7,300 bytes in 484 blocks are definitely lost in loss record 860 of 874
18927 at 0x4024C1C: malloc (vg_replace_malloc.c:195)
18927 by 0x8254B5B: FTPParseRequestCommandLine (app-layer-ftp.c:130)
18927 by 0x822BC32: AppLayerDoParse (app-layer-parser.c:655)
18927 by 0x822C03F: AppLayerDoParse (app-layer-parser.c:689)
18927 by 0x822D3FB: AppLayerParse (app-layer-parser.c:861)
18927 by 0x821FF0E: AppLayerHandleMsg (app-layer.c:174)
18927 by 0x820304F: StreamTcpReassembleProcessAppLayer (stream-tcp-reassemble.c:1911)
18927 by 0x81EDE2B: StreamTcpPacket (stream-tcp.c:2509)
18927 by 0x81EE045: StreamTcp (stream-tcp.c:2527)
18927 by 0x81C9650: TmThreadsSlotVarRun (tm-threads.c:433)
18927 by 0x81C9AE7: TmThreadsSlotVar (tm-threads.c:509)
18927 by 0x40B380D: start_thread (pthread_create.c:300)

Files

0001-app-layer-ftp-fix-mem-leak.patch (719 Bytes) 0001-app-layer-ftp-fix-mem-leak.patch When destroying the ftp state, it was not removing that buffer. Pablo Rincon, 06/01/2010 03:38 PM
Actions #1

Updated by Pablo Rincon almost 14 years ago

When destroying the ftp state, it was not removing that buffer. Attached a patch to fix that.

Actions #2

Updated by Victor Julien almost 14 years ago

Applied, thanks Pablo.

Actions #3

Updated by Victor Julien almost 14 years ago

  • Status changed from Resolved to Closed
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF