svn commit: r320677 - stable/11/sbin/camcontrol

Alan Somers asomers at FreeBSD.org
Wed Jul 5 15:34:07 UTC 2017


Author: asomers
Date: Wed Jul  5 15:34:06 2017
New Revision: 320677
URL: https://svnweb.freebsd.org/changeset/base/320677

Log:
  MFC r319337:
  
  Fix integer overflow in "camcontrol format"
  
  Reported by:	Coverity
  CID:		1011426
  Sponsored by:	Spectra Logic Corp

Modified:
  stable/11/sbin/camcontrol/camcontrol.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/sbin/camcontrol/camcontrol.c
==============================================================================
--- stable/11/sbin/camcontrol/camcontrol.c	Wed Jul  5 15:31:42 2017	(r320676)
+++ stable/11/sbin/camcontrol/camcontrol.c	Wed Jul  5 15:34:06 2017	(r320677)
@@ -6221,15 +6221,15 @@ doreport:
 				if ((scsi_get_sks(sense, ccb->csio.sense_len -
 				     ccb->csio.sense_resid, sks) == 0)
 				 && (quiet == 0)) {
-					int val;
+					uint32_t val;
 					u_int64_t percentage;
 
 					val = scsi_2btoul(&sks[1]);
-					percentage = 10000 * val;
+					percentage = 10000ull * val;
 
 					fprintf(stdout,
 						"\rFormatting:  %ju.%02u %% "
-						"(%d/%d) done",
+						"(%u/%d) done",
 						(uintmax_t)(percentage /
 						(0x10000 * 100)),
 						(unsigned)((percentage /


More information about the svn-src-stable-11 mailing list