|
 |
306fa1 |
autofs-5.1.5 - add NULL check for get_addr_string() return
|
|
|
306fa1 |
|
|
|
306fa1 |
From: Ian Kent <raven@themaw.net>
|
|
|
306fa1 |
|
|
|
306fa1 |
When constructing the mount location string in mount_nfs.c:mount_mount()
|
|
|
306fa1 |
the return from get_addr_string() is not checked for NULL.
|
|
|
306fa1 |
|
|
|
306fa1 |
Signed-off-by: Ian Kent <raven@themaw.net>
|
|
|
306fa1 |
---
|
|
|
306fa1 |
CHANGELOG | 1 +
|
|
|
306fa1 |
modules/mount_nfs.c | 6 ++++++
|
|
|
306fa1 |
2 files changed, 7 insertions(+)
|
|
|
306fa1 |
|
|
|
306fa1 |
--- autofs-5.0.7.orig/CHANGELOG
|
|
|
306fa1 |
+++ autofs-5.0.7/CHANGELOG
|
|
|
306fa1 |
@@ -317,6 +317,7 @@
|
|
|
306fa1 |
- move close stdio descriptors to become_daemon().
|
|
|
306fa1 |
- add systemd service command line option.
|
|
|
306fa1 |
- support strictexpire mount option.
|
|
|
306fa1 |
+- add NULL check for get_addr_string() return.
|
|
|
306fa1 |
|
|
|
306fa1 |
25/07/2012 autofs-5.0.7
|
|
|
306fa1 |
=======================
|
|
|
306fa1 |
--- autofs-5.0.7.orig/modules/mount_nfs.c
|
|
|
306fa1 |
+++ autofs-5.0.7/modules/mount_nfs.c
|
|
|
306fa1 |
@@ -324,7 +324,13 @@ dont_probe:
|
|
|
306fa1 |
socklen_t len = INET6_ADDRSTRLEN;
|
|
|
306fa1 |
char n_buf[len + 1];
|
|
|
306fa1 |
const char *n_addr;
|
|
|
306fa1 |
+
|
|
|
306fa1 |
n_addr = get_addr_string(this->addr, n_buf, len);
|
|
|
306fa1 |
+ if (!n_addr) {
|
|
|
306fa1 |
+ char *estr = strerror_r(errno, buf, MAX_ERR_BUF);
|
|
|
306fa1 |
+ error(ap->logopt, "get_addr_string: %s", estr);
|
|
|
306fa1 |
+ goto forced_fail;
|
|
|
306fa1 |
+ }
|
|
|
306fa1 |
loc = malloc(strlen(n_addr) + strlen(this->path) + 4);
|
|
|
306fa1 |
if (!loc) {
|
|
|
306fa1 |
char *estr = strerror_r(errno, buf, MAX_ERR_BUF);
|