pstore: Fix sprintf format specifier in pstore_dump()
authoralex chen <alex.chen@huawei.com>
Fri, 16 Jan 2015 23:52:03 +0000 (15:52 -0800)
committerTony Luck <tony.luck@intel.com>
Sat, 17 Jan 2015 00:01:29 +0000 (16:01 -0800)
We should use sprintf format specifier "%u" instead of "%d" for
argument of type 'unsigned int' in pstore_dump().

Signed-off-by: Alex Chen <alex.chen@huawei.com>
Reviewed-by: Joseph Qi <joseph.qi@huawei.com>
Acked-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Tony Luck <tony.luck@intel.com>
fs/pstore/platform.c

index 15ee78c5020b24a0ca5c9f9b815157fea4bee96a..c4c9a10c5760e057ef7b019e1013d472a2ccaabb 100644 (file)
@@ -301,7 +301,7 @@ static void pstore_dump(struct kmsg_dumper *dumper,
 
                if (big_oops_buf) {
                        dst = big_oops_buf;
-                       hsize = sprintf(dst, "%s#%d Part%d\n", why,
+                       hsize = sprintf(dst, "%s#%d Part%u\n", why,
                                                        oopscount, part);
                        size = big_oops_buf_sz - hsize;
 
@@ -321,7 +321,7 @@ static void pstore_dump(struct kmsg_dumper *dumper,
                        }
                } else {
                        dst = psinfo->buf;
-                       hsize = sprintf(dst, "%s#%d Part%d\n", why, oopscount,
+                       hsize = sprintf(dst, "%s#%d Part%u\n", why, oopscount,
                                                                        part);
                        size = psinfo->bufsize - hsize;
                        dst += hsize;