lkml.org 
[lkml]   [2019]   [Feb]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 4.14 34/34] isdn: avm: Fix string plus integer warning from Clang
    Date
    From: Nathan Chancellor <natechancellor@gmail.com>

    [ Upstream commit 7afa81c55fca0cad589722cb4bce698b4803b0e1 ]

    A recent commit in Clang expanded the -Wstring-plus-int warning, showing
    some odd behavior in this file.

    drivers/isdn/hardware/avm/b1.c:426:30: warning: adding 'int' to a string does not append to the string [-Wstring-plus-int]
    cinfo->version[j] = "\0\0" + 1;
    ~~~~~~~^~~
    drivers/isdn/hardware/avm/b1.c:426:30: note: use array indexing to silence this warning
    cinfo->version[j] = "\0\0" + 1;
    ^
    & [ ]
    1 warning generated.

    This is equivalent to just "\0". Nick pointed out that it is smarter to
    use "" instead of "\0" because "" is used elsewhere in the kernel and
    can be deduplicated at the linking stage.

    Link: https://github.com/ClangBuiltLinux/linux/issues/309
    Suggested-by: Nick Desaulniers <ndesaulniers@google.com>
    Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/isdn/hardware/avm/b1.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/drivers/isdn/hardware/avm/b1.c b/drivers/isdn/hardware/avm/b1.c
    index b1833d08a5fe..40a099f33bfc 100644
    --- a/drivers/isdn/hardware/avm/b1.c
    +++ b/drivers/isdn/hardware/avm/b1.c
    @@ -423,7 +423,7 @@ void b1_parse_version(avmctrl_info *cinfo)
    int i, j;

    for (j = 0; j < AVM_MAXVERSION; j++)
    - cinfo->version[j] = "\0\0" + 1;
    + cinfo->version[j] = "";
    for (i = 0, j = 0;
    j < AVM_MAXVERSION && i < cinfo->versionlen;
    j++, i += cinfo->versionbuf[i] + 1)
    --
    2.19.1
    \
     
     \ /
      Last update: 2019-02-13 03:47    [W:4.220 / U:0.368 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site