• Christoph Biedl's avatar
    isdn: Partially revert debug format string usage clean up · cee367ad
    Christoph Biedl authored
    commit 19cebbcb upstream.
    
    Commit 35a4a573 ("isdn: clean up debug format string usage") introduced
    a safeguard to avoid accidential format string interpolation of data
    when calling debugl1 or HiSax_putstatus. This did however not take into
    account VHiSax_putstatus (called by HiSax_putstatus) does *not* call
    vsprintf if the head parameter is NULL - the format string is treated
    as plain text then instead. As a result, the string "%s" is processed
    literally, and the actual information is lost. This affects the isdnlog
    userspace program which stopped logging information since that commit.
    
    So revert the HiSax_putstatus invocations to the previous state.
    
    Fixes: 35a4a573 ("isdn: clean up debug format string usage")
    Cc: Kees Cook <keescook@chromium.org>
    Cc: Karsten Keil <isdn@linux-pingi.de>
    Signed-off-by: default avatarChristoph Biedl <linux-kernel.bfrz@manchmal.in-ulm.de>
    Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
    Signed-off-by: default avatarKamal Mostafa <kamal@canonical.com>
    cee367ad
q931.c 30.8 KB