From 00601179625ffb71461bbcc46629791895cf4793 Mon Sep 17 00:00:00 2001 From: Simon Wilkinson Date: Wed, 20 Feb 2013 14:54:55 +0000 Subject: [PATCH] volser: Make it clear what's copied from partList XVolListPartitions uses partList to maintain a list of partition IDs - it does not use the partFlags field. Make it clear when we populate the partEntries field that we are only copying the partition list. Caught by coverity (#986030) Change-Id: I8c46ee5c220f0cb2b65943ad5faa80877ca46685 Reviewed-on: http://gerrit.openafs.org/9262 Reviewed-by: Jeffrey Altman Tested-by: BuildBot Reviewed-by: Derrick Brashear --- src/volser/volprocs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/volser/volprocs.c b/src/volser/volprocs.c index 2fed208b3..17ae45259 100644 --- a/src/volser/volprocs.c +++ b/src/volser/volprocs.c @@ -1892,7 +1892,7 @@ XVolListPartitions(struct rx_call *acid, struct partEntries *pEntries) pEntries->partEntries_val = malloc(j * sizeof(int)); if (!pEntries->partEntries_val) return ENOMEM; - memcpy((char *)pEntries->partEntries_val, (char *)&partList, + memcpy(pEntries->partEntries_val, partList.partId, j * sizeof(int)); pEntries->partEntries_len = j; } else { -- 2.39.5