isdn: Constify some function parameters
authorEmese Revfy <re.emese@gmail.com>
Fri, 16 Dec 2016 21:40:47 +0000 (13:40 -0800)
committerDavid S. Miller <davem@davemloft.net>
Sun, 18 Dec 2016 02:46:35 +0000 (21:46 -0500)
The coming initify gcc plugin expects const pointer types, and caught
some __printf arguments that weren't const yet. This fixes those.

Signed-off-by: Emese Revfy <re.emese@gmail.com>
[kees: expanded commit message]
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/isdn/hisax/config.c
drivers/isdn/hisax/hisax.h

index bf04d2a3cf4afe4614765b2b3d08d32ec77571a6..2d12c6ceeb89ae08bb6c100fd77320386010a043 100644 (file)
@@ -659,7 +659,7 @@ int jiftime(char *s, long mark)
 
 static u_char tmpbuf[HISAX_STATUS_BUFSIZE];
 
-void VHiSax_putstatus(struct IsdnCardState *cs, char *head, char *fmt,
+void VHiSax_putstatus(struct IsdnCardState *cs, char *head, const char *fmt,
                      va_list args)
 {
        /* if head == NULL the fmt contains the full info */
@@ -669,23 +669,24 @@ void VHiSax_putstatus(struct IsdnCardState *cs, char *head, char *fmt,
        u_char          *p;
        isdn_ctrl       ic;
        int             len;
+       const u_char    *data;
 
        if (!cs) {
                printk(KERN_WARNING "HiSax: No CardStatus for message");
                return;
        }
        spin_lock_irqsave(&cs->statlock, flags);
-       p = tmpbuf;
        if (head) {
+               p = tmpbuf;
                p += jiftime(p, jiffies);
                p += sprintf(p, " %s", head);
                p += vsprintf(p, fmt, args);
                *p++ = '\n';
                *p = 0;
                len = p - tmpbuf;
-               p = tmpbuf;
+               data = tmpbuf;
        } else {
-               p = fmt;
+               data = fmt;
                len = strlen(fmt);
        }
        if (len > HISAX_STATUS_BUFSIZE) {
@@ -699,13 +700,12 @@ void VHiSax_putstatus(struct IsdnCardState *cs, char *head, char *fmt,
        if (i >= len)
                i = len;
        len -= i;
-       memcpy(cs->status_write, p, i);
+       memcpy(cs->status_write, data, i);
        cs->status_write += i;
        if (cs->status_write > cs->status_end)
                cs->status_write = cs->status_buf;
-       p += i;
        if (len) {
-               memcpy(cs->status_write, p, len);
+               memcpy(cs->status_write, data + i, len);
                cs->status_write += len;
        }
 #ifdef KERNELSTACK_DEBUG
@@ -729,7 +729,7 @@ void VHiSax_putstatus(struct IsdnCardState *cs, char *head, char *fmt,
        }
 }
 
-void HiSax_putstatus(struct IsdnCardState *cs, char *head, char *fmt, ...)
+void HiSax_putstatus(struct IsdnCardState *cs, char *head, const char *fmt, ...)
 {
        va_list args;
 
index 6ead6314e6d25977e84902dba491da94610a96ae..338d0408b3778a87992825928a13106ebb18b9a4 100644 (file)
@@ -1288,9 +1288,9 @@ int jiftime(char *s, long mark);
 int HiSax_command(isdn_ctrl *ic);
 int HiSax_writebuf_skb(int id, int chan, int ack, struct sk_buff *skb);
 __printf(3, 4)
-void HiSax_putstatus(struct IsdnCardState *cs, char *head, char *fmt, ...);
+void HiSax_putstatus(struct IsdnCardState *cs, char *head, const char *fmt, ...);
 __printf(3, 0)
-void VHiSax_putstatus(struct IsdnCardState *cs, char *head, char *fmt, va_list args);
+void VHiSax_putstatus(struct IsdnCardState *cs, char *head, const char *fmt, va_list args);
 void HiSax_reportcard(int cardnr, int sel);
 int QuickHex(char *txt, u_char *p, int cnt);
 void LogFrame(struct IsdnCardState *cs, u_char *p, int size);