commit | a37383db8081b66eafc3860ede23edbfd39ac8ad | [log] [tgz] |
---|---|---|
author | Vladimir Serbinenko <phcoder@gmail.com> | Tue Nov 26 02:41:26 2013 +0100 |
committer | Vladimir Serbinenko <phcoder@gmail.com> | Fri Jan 10 18:08:16 2014 +0100 |
tree | 3f793f4b2342aa440efe3fa649244ec064ced5d8 | |
parent | 4b5012a4a2bf804d395ae3a72c76f50c4a9189db [diff] [blame] |
Replace all occurences of sprintf with snprintf THis reduces risks of bufer overflows. Change-Id: I77f80e76efec16ac0a0af83d76430a8126a7602d Signed-off-by: Vladimir Serbinenko <phcoder@gmail.com> Reviewed-on: http://review.coreboot.org/4279 Tested-by: build bot (Jenkins) Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
diff --git a/src/mainboard/intel/wtm2/i915.c b/src/mainboard/intel/wtm2/i915.c index df88138..b664c35 100644 --- a/src/mainboard/intel/wtm2/i915.c +++ b/src/mainboard/intel/wtm2/i915.c
@@ -88,7 +88,7 @@ static char *regname(unsigned long addr) { static char name[16]; - sprintf(name, "0x%lx", addr); + snprintf(name, sizeof (name), "0x%lx", addr); return name; }