diff options
author | Jeremy Bettis <jbettis@google.com> | 2021-09-20 16:08:14 -0600 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-09-24 16:13:29 +0000 |
commit | a4056238da1d6bc6327b8d090845db9a44fe5f34 (patch) | |
tree | 184b992f1f674f0c55e862801cccdebcc6cc167f /util/genvif.c | |
parent | 2caf7b2cadb5dd12bee3442565f02abfb76f9077 (diff) | |
download | chrome-ec-a4056238da1d6bc6327b8d090845db9a44fe5f34.tar.gz |
ec: Fail on uninitialized variables
The zephyr toolchain has -Werror=maybe-uninitialized enabled by default,
so enable it for all gcc EC builds to prevent possible uninitialized
vars from sneaking into the codebase. For clang builds, use
-Werror=uninitialized instead.
BRANCH=None
BUG=None
TEST=in docker: make -j CRYPTOC_DIR="${MODULES_DIR}/cryptoc" \
FTDIVERSION=1 HOSTGCOV='gcov' \
CROSS_COMPILE_arm=/opt/zephyr-sdk/arm-zephyr-eabi/bin/arm-zephyr-eabi- \
coverage
in chroot; make try_build_boards buildall_only
Change-Id: I3efb4b2958f2037606195743d768870e1d0b18b9
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3171939
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Auto-Submit: Jeremy Bettis <jbettis@chromium.org>
Commit-Queue: Jack Rosenthal <jrosenth@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Diffstat (limited to 'util/genvif.c')
-rw-r--r-- | util/genvif.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/util/genvif.c b/util/genvif.c index a8b7f24ec4..6722df9349 100644 --- a/util/genvif.c +++ b/util/genvif.c @@ -2791,7 +2791,7 @@ static void init_vif_component_general_pd_fields( * to NO, then this field shall be set to YES. */ { - bool supports_to_dfp; + bool supports_to_dfp = false; switch (type) { case SRC: @@ -2833,7 +2833,7 @@ static void init_vif_component_general_pd_fields( * 5 (eMarker) then this field shall be ignored by Testers. */ { - bool supports_to_ufp; + bool supports_to_ufp = false; switch (type) { case SRC: |