mISDN: Don't try to print a sockptr_t from debug logging code.
authorDavid S. Miller <davem@davemloft.net>
Fri, 24 Jul 2020 23:36:13 +0000 (16:36 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 24 Jul 2020 23:36:13 +0000 (16:36 -0700)
drivers/isdn/mISDN/socket.c: In function ‘data_sock_setsockopt’:
./include/linux/kern_levels.h:5:18: warning: format ‘%p’ expects argument of type ‘void *’, but argument 6 has type ‘sockptr_t’ [-Wformat=]
    5 | #define KERN_SOH "\001"  /* ASCII Start Of Header */
      |                  ^~~~~~
./include/linux/kern_levels.h:15:20: note: in expansion of macro ‘KERN_SOH’
   15 | #define KERN_DEBUG KERN_SOH "7" /* debug-level messages */
      |                    ^~~~~~~~
drivers/isdn/mISDN/socket.c:410:10: note: in expansion of macro ‘KERN_DEBUG’
  410 |   printk(KERN_DEBUG "%s(%p, %d, %x, %p, %d)\n", __func__, sock,
      |          ^~~~~~~~~~
drivers/isdn/mISDN/socket.c:410:38: note: format string is defined here
  410 |   printk(KERN_DEBUG "%s(%p, %d, %x, %p, %d)\n", __func__, sock,
      |                                     ~^
      |                                      |
      |                                      void *

Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/isdn/mISDN/socket.c

index 2835daae9e9f3aed0e7196f118f86167ad9ee0a4..a6606736d8c5033aeccf48d2692e63832796a9fc 100644 (file)
@@ -407,8 +407,8 @@ static int data_sock_setsockopt(struct socket *sock, int level, int optname,
        int err = 0, opt = 0;
 
        if (*debug & DEBUG_SOCKET)
-               printk(KERN_DEBUG "%s(%p, %d, %x, %p, %d)\n", __func__, sock,
-                      level, optname, optval, len);
+               printk(KERN_DEBUG "%s(%p, %d, %x, optval, %d)\n", __func__, sock,
+                      level, optname, len);
 
        lock_sock(sk);