Commit Diff
Diff:
c9b152fa4119bc0f4ca27a2e163840a1e2dfbbc5
7faa3ed7d6613f23e998604461f543e763278f7e
7faa3ed7d6613f23e998604461f543e763278f7e
Commit:
7faa3ed7d6613f23e998604461f543e763278f7e
Tree:
c83214c572388de0652a81262327f482b7ab9ed4
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
--- 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;
}