Closed Bug 334913 Opened 20 years ago Closed 20 years ago

jar_parse_any leaks mfdig = jar_digest_section because ADDITEM macro returns

Categories

(NSS :: Libraries, defect, P3)

3.11
All
Linux
defect

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 337354
3.11.2

People

(Reporter: timeless, Unassigned)

References

()

Details

(Keywords: coverity, memory-leak, Whiteboard: [CID 753])

found by coverity 68 #define ADDITEM(list,type,pathname,data,size) \ 69 { int err; err = jar_append (list, type, pathname, data, size); \ 70 if (err < 0) return err; } more amusingly: 407 char x_name [SZ], x_md5 [SZ], x_sha [SZ]; At conditional (3): "&x_name == 0" taking false path 641 if(!x_name || !*x_name) { yes, we can be certain it will always take the false path here. would someone please be so kind as to put that useless code out of its misery? :)
Priority: -- → P3
Hardware: PC → All
Target Milestone: --- → 3.11.2
consolidating all jarver.c memory leaks into one bug: bug 337354 *** This bug has been marked as a duplicate of 337354 ***
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → DUPLICATE
Coverity CID 753
Whiteboard: [CID 753]
You need to log in before you can comment on or make changes to this bug.