diff options
| author | stefanct <stefanct@2b7e53f0-3cfb-0310-b3e9-8179ed1497e1> | 2014-05-26 22:05:31 +0000 |
|---|---|---|
| committer | stefanct <stefanct@2b7e53f0-3cfb-0310-b3e9-8179ed1497e1> | 2014-05-26 22:05:31 +0000 |
| commit | 034f719ab77b222f130f564f63b948982ce95dd0 (patch) | |
| tree | 4240d774ca35015fb68fb32a2f2c2b0a2ca4b1ec /flashchips.c | |
| parent | cfc7fc504858b4d32b6c0445e9ab94a676bfa970 (diff) | |
| download | flashrom-034f719ab77b222f130f564f63b948982ce95dd0.tar.gz | |
Fix selfcheck of various arrays.
Stefan Reinauer has reported ridiculous NULL checks for arrays in our
self_check function found by Coverity (CID1130005). This patch removes
the useless checks but keeps and fixes the one responsible for the
flashchips array by exporting the array size in a new constant.
Signed-off-by: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
git-svn-id: https://code.coreboot.org/svn/flashrom/trunk@1799 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
Diffstat (limited to 'flashchips.c')
| -rw-r--r-- | flashchips.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/flashchips.c b/flashchips.c index 027b996..a082e75 100644 --- a/flashchips.c +++ b/flashchips.c @@ -13424,5 +13424,7 @@ const struct flashchip flashchips[] = { .write = NULL, }, - { NULL } + {0} }; + +const unsigned int flashchips_size = ARRAY_SIZE(flashchips); |