Commit Diff
Diff:
c9b152fa4119bc0f4ca27a2e163840a1e2dfbbc5
7faa3ed7d6613f23e998604461f543e763278f7e
Commit:
7faa3ed7d6613f23e998604461f543e763278f7e
Tree:
c83214c572388de0652a81262327f482b7ab9ed4
Author:
Alexander Barton <alex@barton.de>
Committer:
Alexander Barton <alex@barton.de>
Date:
Tue May 22 11:31:08 2012 UTC
Message:
Pidfile_Create(): Don't leak file descriptor on error path Detected by cppcheck: [src/ngircd/ngircd.c:502]: (error) Resource leak: pidfd
blob - f28ff05bcfb778065dc99a50e7459472eee8a68f
blob + 8a93bcb00839a796b262b890d12b373d662ccf56
--- src/ngircd/ngircd.c
+++ src/ngircd/ngircd.c
@@ -498,7 +498,8 @@ Pidfile_Create(pid_t pid)
len = snprintf(pidbuf, sizeof pidbuf, "%ld\n", (long)pid);
if (len < 0 || len >= (int)sizeof pidbuf) {
- Log( LOG_ERR, "Error converting pid");
+ Log(LOG_ERR, "Error converting pid");
+ close(pidfd);
return;
}
IRCNow