Commit
18f41f2fa2d021f7baad37ba6fbe01b49309afbb
Exists in
smarc_8mq_lf_v2020.04
and in
18 other branches
8qm-imx_v2020.04_5.4.70_2.3.0, emb_lf_v2022.04, emb_lf_v2023.04, pitx_8mp_lf_v2020.04, smarc-8m-android-10.0.0_2.6.0, smarc-8m-android-11.0.0_2.0.0, smarc-8mp-android-11.0.0_2.0.0, smarc-imx6_v2018.03_4.14.98_2.0.0_ga, smarc-imx7_v2018.03_4.14.98_2.0.0_ga, smarc-imx_v2018.03_4.14.78_1.0.0_ga, smarc_8m-imx_v2018.03_4.14.98_2.0.0_ga, smarc_8m-imx_v2019.04_4.19.35_1.1.0, smarc_8m_00d0-imx_v2018.03_4.14.98_2.0.0_ga, smarc_8mm-imx_v2018.03_4.14.98_2.0.0_ga, smarc_8mm-imx_v2019.04_4.19.35_1.1.0, smarc_8mm-imx_v2020.04_5.4.24_2.1.0, smarc_8mp_lf_v2020.04, smarc_8mq-imx_v2020.04_5.4.24_2.1.0
cmd: ubi: remove unnecessary logical constraint
A size_t variable can never be negative.
The problem was indicated by cppcheck.
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Reviewed-by: Tom Rini <trini@konsulko.com>
Reviewed-by: Heiko Schocher <hs@denx.de>
Showing
1 changed file
with
1 additions
and
1 deletions
Side-by-side Diff
... |
... |
@@ -308,7 +308,7 @@ |
308
|
308 |
return ENODEV; |
309
|
309 |
|
310
|
310 |
rsvd_bytes = vol->reserved_pebs * (ubi->leb_size - vol->data_pad); |
311
|
|
- if (size < 0 || size > rsvd_bytes) { |
|
311 |
+ if (size > rsvd_bytes) { |
312
|
312 |
printf("size > volume size! Aborting!\n"); |
313
|
313 |
return EINVAL; |
314
|
314 |
} |