diff options
author | Martin Roth <martinroth@google.com> | 2018-11-26 18:03:32 -0700 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2018-11-28 11:47:32 +0000 |
commit | 7ed3fe2b472ce39883c79bffacccc3221d6c2b38 (patch) | |
tree | 1f91edb5f0cdf978e64f1fa9e6b2c9652367c960 /src/lib/memset.c | |
parent | d97eb646e75e33c5a4068c21fb20b75ef2acb820 (diff) |
romcc: Increase buffer size to fit max string size
On the updated builder image, the build is failing because the system
compiler has been updated to GCC 8.2.0. It complains about the
possibility of overflow when putting one 30 character buffer plus 2
characters into another 30 character buffer. To fix this, increase
the recipient buffer size by 2.
romcc.c:3645:2: note: 'sprintf' output between 3 and 32 bytes into a destination
of size 30 [-Werror=format-overflow=]
sprintf(buf, "\"%s\"", scratch);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
romcc.c:3649:2: note: 'sprintf' output between 3 and 32 bytes into a destination
of size 30 [-Werror=format-overflow=]
sprintf(buf, "\"%s\"", scratch);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Change-Id: I7879a7202cc3ff52301b10118fc49fcc601f133e
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/c/29850
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Rudolph <siro@das-labor.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Diffstat (limited to 'src/lib/memset.c')
0 files changed, 0 insertions, 0 deletions