From 2c9d9aaf80619dcc4cb49ad2063aa9a979a3ed4c Mon Sep 17 00:00:00 2001 From: Alex Tereschenko Date: Sun, 28 Jan 2018 16:16:36 +0100 Subject: [PATCH] intel_edison_fab_c.c: use snprintf(), not sprintf(), to avoid overflows Signed-off-by: Alex Tereschenko --- src/x86/intel_edison_fab_c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/x86/intel_edison_fab_c.c b/src/x86/intel_edison_fab_c.c index be96838..3f612ec 100644 --- a/src/x86/intel_edison_fab_c.c +++ b/src/x86/intel_edison_fab_c.c @@ -147,7 +147,7 @@ mraa_intel_edison_pinmode_change(int sysfs, int mode) mraa_result_t ret = MRAA_SUCCESS; char mode_buf[MAX_MODE_SIZE]; - int length = sprintf(mode_buf, "%s%u", useDebugFS ? "mode" : "", mode); + int length = snprintf(mode_buf, MAX_MODE_SIZE, "%s%u", useDebugFS ? "mode" : "", mode); if (write(modef, mode_buf, length * sizeof(char)) == -1) { ret = MRAA_ERROR_INVALID_RESOURCE; }