Actions
Bug #1185
closednapatech: cppcheck warning
Affected Versions:
Effort:
Difficulty:
Label:
Description
Using cppcheck 1.64:
[src/runmode-napatech.c:171]: (error) Memory leak: conf
Updated by Andreas Moe over 9 years ago
Can't seem to find this cppcheck error message, is this bug report not an issue anymore?
Updated by Victor Julien over 9 years ago
It's quite easy to spot w/o cppcheck:
struct NapatechStreamDevConf *conf = SCMalloc(sizeof(struct NapatechStreamDevConf));
if (unlikely(conf == NULL))
return NULL;
if (dev_len < 3 || dev_len > 5)
{
SCLogError(SC_ERR_NAPATECH_PARSE_CONFIG, "Could not parse config for device: %s - invalid length", device);
return NULL; // <- conf not freed.
}
Updated by Victor Julien about 8 years ago
- Status changed from New to Assigned
- Assignee changed from Anonymous to Victor Julien
- Target version changed from TBD to 4.0.2/4.0.3
Updated by Victor Julien about 8 years ago
- Status changed from Assigned to Closed
Actions