diff mbox

ocfs2: fix snprintf format specifier in dlmdebug.c

Message ID 54AE4E6D.6050902@huawei.com
State New, archived
Headers show

Commit Message

Alex Chen Jan. 8, 2015, 9:31 a.m. UTC
Use snprintf format specifier "%lu" instead of "%ld" for
argument of type 'unsigned long'.

Signed-off-by: Alex Chen <alex.chen@huawei.com>
Reviewed-by: Joseph Qi <joseph.qi@huawei.com>
---
 fs/ocfs2/dlm/dlmdebug.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Mark Fasheh Jan. 16, 2015, 11:26 p.m. UTC | #1
On Thu, Jan 08, 2015 at 05:31:25PM +0800, alex chen wrote:
> Use snprintf format specifier "%lu" instead of "%ld" for
> argument of type 'unsigned long'.
> 
> Signed-off-by: Alex Chen <alex.chen@huawei.com>

Reviewed-by: Mark Fasheh <mfasheh@suse.de>


> Reviewed-by: Joseph Qi <joseph.qi@huawei.com>

Btw, did this review happen on the mailing list? It's customary for review
to happen publicly.

Thanks,
	--Mark

--
Mark Fasheh
diff mbox

Patch

diff --git a/fs/ocfs2/dlm/dlmdebug.c b/fs/ocfs2/dlm/dlmdebug.c
index 149eb55..8251360 100644
--- a/fs/ocfs2/dlm/dlmdebug.c
+++ b/fs/ocfs2/dlm/dlmdebug.c
@@ -406,7 +406,7 @@  static int debug_purgelist_print(struct dlm_ctxt *dlm, char *buf, int len)
 	}
 	spin_unlock(&dlm->spinlock);

-	out += snprintf(buf + out, len - out, "Total on list: %ld\n", total);
+	out += snprintf(buf + out, len - out, "Total on list: %lu\n", total);

 	return out;
 }
@@ -464,7 +464,7 @@  static int debug_mle_print(struct dlm_ctxt *dlm, char *buf, int len)
 	spin_unlock(&dlm->master_lock);

 	out += snprintf(buf + out, len - out,
-			"Total: %ld, Longest: %ld\n", total, longest);
+			"Total: %lu, Longest: %lu\n", total, longest);
 	return out;
 }