From bf4b2fd3e26dcf2a0db704815e05d77a558d38c6 Mon Sep 17 00:00:00 2001 From: Garrett Wollman Date: Mon, 8 Aug 2011 23:26:38 -0400 Subject: [PATCH] bos: don't dereference a null pointer when printing an error message The parameter we are interested in is at MRAFS_OFFSET + 17, not MRAFS_OFFSET + 13. Change-Id: Ib856ff40c5949cde95a2b277cd44253b87c3c2a4 Found-by: clang static analyzer Reviewed-on: http://gerrit.openafs.org/5178 Tested-by: BuildBot Reviewed-by: Simon Wilkinson Reviewed-by: Derrick Brashear --- src/bozo/bos.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/bozo/bos.c b/src/bozo/bos.c index 5d74868bc..e108f225e 100644 --- a/src/bozo/bos.c +++ b/src/bozo/bos.c @@ -1566,7 +1566,7 @@ SalvageCmd(struct cmd_syndesc *as, void *arock) &mrafsParm.OptResidencies); if (code) { printf("bos: '%s' is not a valid residency mask.\n", - as->parms[MRAFS_OFFSET + 13].items->data); + as->parms[MRAFS_OFFSET + 17].items->data); return code; } } -- 2.39.5