Open Bug 1142672 Opened 5 years ago Updated 2 years ago
[logalloc] Leak of |fd| in |replace
Coverity indicates a leak of |fd| if |open| returns 0 .  https://hg.mozilla.org/mozilla-central/annotate/0190a1d17294/memory/replace/logalloc/LogAlloc.cpp#l118
The only way this could happen is if there is no stdin, and I'm pretty sure many other things would break in that case. If someone wants to fix this, be my guest, but a theoretical file descriptor leak on a runtime opt-in tool is not a high priority.
Whiteboard: [MemShrink][CID 1286471] → [CID 1286471]
You make a reasonable point that this probably isn't high priority, but it also took you 251X more characters in your response than to fix the bug at hand. Perhaps this would be a good candidate for a mentor bug?
You need to log in before you can comment on or make changes to this bug.