diff options
| author | Karel Zak | 2013-03-27 16:08:47 +0100 |
|---|---|---|
| committer | Karel Zak | 2013-03-27 16:08:47 +0100 |
| commit | 8362545b4a3dc1b2903b7aa2e40eaf472aec130c (patch) | |
| tree | 72c94c0b0005a03e0ff6e9fa1e7211b2e6ffb776 /term-utils/agetty.c | |
| parent | libblkid: (udf) fix possible null pointer dereference [coverity scan] (diff) | |
| download | kernel-qcow2-util-linux-8362545b4a3dc1b2903b7aa2e40eaf472aec130c.tar.gz kernel-qcow2-util-linux-8362545b4a3dc1b2903b7aa2e40eaf472aec130c.tar.xz kernel-qcow2-util-linux-8362545b4a3dc1b2903b7aa2e40eaf472aec130c.zip | |
lib/xalloc: fix mamory leak in xgethostname() [coverity scan]
Signed-off-by: Karel Zak <kzak@redhat.com>
Diffstat (limited to 'term-utils/agetty.c')
| -rw-r--r-- | term-utils/agetty.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/term-utils/agetty.c b/term-utils/agetty.c index 7ac12f201..f745e6800 100644 --- a/term-utils/agetty.c +++ b/term-utils/agetty.c @@ -1209,9 +1209,10 @@ static char *xgethostname(void) if (!name) log_err(_("failed to allocate memory: %m")); - if (gethostname(name, sz) != 0) + if (gethostname(name, sz) != 0) { + free(name); return NULL; - + } name[sz - 1] = '\0'; return name; } @@ -1226,9 +1227,10 @@ static char *xgetdomainname(void) if (!name) log_err(_("failed to allocate memory: %m")); - if (getdomainname(name, sz) != 0) + if (getdomainname(name, sz) != 0) { + free(name); return NULL; - + } name[sz - 1] = '\0'; return name; #endif |