[PATCH] sysfs.txt: update show method notes about sprintf/snprintf/scnprintf usage
From: Seymour, Shane M <hidden>
Date: 2015-06-25 00:55:26
Changed the documentation to allow sprintf() for small single values and explicitly say snprintf() must never be used in a show function to format data to be returned to user space. Change based on a discussion about the patch st: convert DRIVER_ATTR macros to DRIVER_ATTR_RO Suggested-by: Greg Kroah-Hartman <redacted> Signed-off-by: Shane Seymour <redacted> ---
--- a/Documentation/filesystems/sysfs.txt 2015-06-22 14:18:40.278620871 -0500
+++ b/Documentation/filesystems/sysfs.txt 2015-06-24 13:42:21.344446532 -0500@@ -212,7 +212,9 @@ Other notes: - show() methods should return the number of bytes printed into the buffer. This is the return value of scnprintf(). -- show() should always use scnprintf(). +- show() must not use snprintf() when formatting a value to be + returned to user space. For small single values you can use + sprintf() otherwise you must use scnprintf(). - store() should return the number of bytes used from the buffer. If the entire buffer has been used, just return the count argument.