lkml.org 
[lkml]   [2019]   [Feb]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[PATCH 2/2] test_firmware: silence underflow warning in test_dev_config_update_u8()
    We put an upper bound on "new" but we don't check for negatives. In
    this case the underflow doesn't matter very much, but we may as well
    make the static checker happy.

    Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
    ---
    lib/test_firmware.c | 7 ++-----
    1 file changed, 2 insertions(+), 5 deletions(-)

    diff --git a/lib/test_firmware.c b/lib/test_firmware.c
    index 7222093ee00b..5911b0f1a715 100644
    --- a/lib/test_firmware.c
    +++ b/lib/test_firmware.c
    @@ -326,15 +326,12 @@ static ssize_t test_dev_config_show_int(char *buf, int cfg)
    static int test_dev_config_update_u8(const char *buf, size_t size, u8 *cfg)
    {
    int ret;
    - long new;
    + u8 new;

    - ret = kstrtol(buf, 10, &new);
    + ret = kstrtou8(buf, 10, &new);
    if (ret)
    return ret;

    - if (new > U8_MAX)
    - return -EINVAL;
    -
    mutex_lock(&test_fw_mutex);
    *(u8 *)cfg = new;
    mutex_unlock(&test_fw_mutex);
    --
    2.17.1
    \
     
     \ /
      Last update: 2019-02-21 19:41    [W:3.140 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site