commit 3365ec8a073443e09d12c0c701b33c0c844f6362
author Patrick Williams <patrick@stwcx.xyz> Wed Aug 17 17:45:18 2016 -0500
committer Patrick Williams <patrick@stwcx.xyz> Thu Aug 18 17:32:39 2016 +0000
tree a985b1147886c978d6806ee94f3cd34a11297830
parent 867da9714222e5d6babdad930d7238486e4698ca

Fix warnings in printf format

A number of arguments to printf were 'size_t', which is the
same size as 'int' only on 32-bit architectures.  Replace all
%d format strings with %zd where size_t is the type passed.

Change-Id: Ida863312d692f47ca8240f32b75b8c55a8accc19
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

writefrudata.C

diff --git a/writefrudata.C b/writefrudata.C
index 8a34e6e..035a0c7 100644
--- a/writefrudata.C
+++ b/writefrudata.C
@@ -479,7 +479,7 @@
         {
             // Our file size is less than what it needs to be. +2 because we are
             // using area len that is at 2 byte off area_offset
-            fprintf(stderr, "fru file is incomplete. Size:[%d]\n",data_len);
+            fprintf(stderr, "fru file is incomplete. Size:[%zd]\n",data_len);
             return rc;
         }
         else if(area_offset)
@@ -492,14 +492,14 @@
             size_t  area_len = area_hdr[1] * IPMI_EIGHT_BYTES;
             uint8_t area_data[area_len] = {0};
 
-            printf("fru data size:[%d], area offset:[%d], area_size:[%d]\n",
+            printf("fru data size:[%zd], area offset:[%zd], area_size:[%zd]\n",
                     data_len, area_offset, area_len);
 
             // See if we really have that much buffer. We have area offset amd
             // from there, the actual len.
             if(data_len < (area_len + area_offset))
             {
-                fprintf(stderr, "Incomplete Fru file.. Size:[%d]\n",data_len);
+                fprintf(stderr, "Incomplete Fru file.. Size:[%zd]\n",data_len);
                 return rc;
             }
 
@@ -510,12 +510,12 @@
             rc = verify_fru_data(area_data, area_len);
             if(rc < 0)
             {
-                fprintf(stderr, "Error validating fru area. offset:[%d]\n",area_offset);
+                fprintf(stderr, "Error validating fru area. offset:[%zd]\n",area_offset);
                 return rc;
             }
             else
             {
-                printf("Successfully verified area checksum. offset:[%d]\n",area_offset);
+                printf("Successfully verified area checksum. offset:[%zd]\n",area_offset);
             }
 
             // We already have a vector that is passed to us containing all
@@ -553,7 +553,7 @@
     }
     else
     {
-        fprintf(stderr, "Incomplete fru data file. Size:[%d]\n", data_len);
+        fprintf(stderr, "Incomplete fru data file. Size:[%zd]\n", data_len);
         return rc;
     }
 
@@ -734,7 +734,7 @@
     bytes_read = fread(fru_data, data_len, 1, fru_fp);
     if(bytes_read != 1)
     {
-        fprintf(stderr, "Failed reading fru data. Bytes_read=[%d]\n",bytes_read);
+        fprintf(stderr, "Failed reading fru data. Bytes_read=[%zd]\n",bytes_read);
         perror("Error:");
         return cleanup_error(fru_fp, fru_area_vec);
     }