From 3ad7ba106dea28277b9e1f7a7370ba17b9fb3d17 Mon Sep 17 00:00:00 2001 From: Simon Wilkinson Date: Tue, 26 Feb 2013 11:55:32 +0000 Subject: [PATCH] fs: Catch pioctl failure in mkmount If the VIOC_FILE_CELL_NAME pioctl fails in fs mkmount, return an error to the caller, instead of ploughing on with potentially bad data. Caught by coverity (#988418) Change-Id: I68df5dd26d955db860dd016c76109f7d262bad17 Reviewed-on: http://gerrit.openafs.org/9267 Tested-by: BuildBot Reviewed-by: Chas Williams - CONTRACTOR Reviewed-by: Derrick Brashear --- src/venus/fs.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/venus/fs.c b/src/venus/fs.c index 5f7e968e1..d8dc4598c 100644 --- a/src/venus/fs.c +++ b/src/venus/fs.c @@ -1931,8 +1931,13 @@ defect #3069 blob.in_size = 0; blob.out_size = AFS_PIOCTL_MAXSIZE; blob.out = space; - pioctl(Parent(as->parms[0].items->data), VIOC_FILE_CELL_NAME, - &blob, 1); + code = pioctl(Parent(as->parms[0].items->data), VIOC_FILE_CELL_NAME, + &blob, 1); + if (code) { + fprintf(stderr, + "%s: couldn't get cell name for file's parent\n", pn); + return 1; + } } dir = afsconf_Open(AFSDIR_CLIENT_ETC_DIRPATH); -- 2.39.5