diff options
author | Kees Cook <keescook@chromium.org> | 2013-09-13 17:52:04 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-09-13 20:02:50 -0400 |
commit | 35a4a5733b0a8290de39558b82896ab795b108a7 (patch) | |
tree | a4a1d7cc33205a1acb098cdba96e319ada715be6 /drivers/isdn/hisax/w6692.c | |
parent | a2a69f0b35762410c4194f9827354310f68470be (diff) |
isdn: clean up debug format string usage
Avoid unneeded local string buffers for constructing debug output. Also
cleans up debug calls that contain a single parameter so that they cannot
be accidentally parsed as format strings.
Signed-off-by: Kees Cook <keescook@chromium.org>
Cc: Karsten Keil <isdn@linux-pingi.de>
Cc: David Miller <davem@davemloft.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/isdn/hisax/w6692.c')
-rw-r--r-- | drivers/isdn/hisax/w6692.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/isdn/hisax/w6692.c b/drivers/isdn/hisax/w6692.c index d8cac6935818..a85895585d90 100644 --- a/drivers/isdn/hisax/w6692.c +++ b/drivers/isdn/hisax/w6692.c | |||
@@ -154,7 +154,7 @@ W6692_empty_fifo(struct IsdnCardState *cs, int count) | |||
154 | 154 | ||
155 | t += sprintf(t, "W6692_empty_fifo cnt %d", count); | 155 | t += sprintf(t, "W6692_empty_fifo cnt %d", count); |
156 | QuickHex(t, ptr, count); | 156 | QuickHex(t, ptr, count); |
157 | debugl1(cs, cs->dlog); | 157 | debugl1(cs, "%s", cs->dlog); |
158 | } | 158 | } |
159 | } | 159 | } |
160 | 160 | ||
@@ -196,7 +196,7 @@ W6692_fill_fifo(struct IsdnCardState *cs) | |||
196 | 196 | ||
197 | t += sprintf(t, "W6692_fill_fifo cnt %d", count); | 197 | t += sprintf(t, "W6692_fill_fifo cnt %d", count); |
198 | QuickHex(t, ptr, count); | 198 | QuickHex(t, ptr, count); |
199 | debugl1(cs, cs->dlog); | 199 | debugl1(cs, "%s", cs->dlog); |
200 | } | 200 | } |
201 | } | 201 | } |
202 | 202 | ||
@@ -226,7 +226,7 @@ W6692B_empty_fifo(struct BCState *bcs, int count) | |||
226 | t += sprintf(t, "W6692B_empty_fifo %c cnt %d", | 226 | t += sprintf(t, "W6692B_empty_fifo %c cnt %d", |
227 | bcs->channel + '1', count); | 227 | bcs->channel + '1', count); |
228 | QuickHex(t, ptr, count); | 228 | QuickHex(t, ptr, count); |
229 | debugl1(cs, bcs->blog); | 229 | debugl1(cs, "%s", bcs->blog); |
230 | } | 230 | } |
231 | } | 231 | } |
232 | 232 | ||
@@ -264,7 +264,7 @@ W6692B_fill_fifo(struct BCState *bcs) | |||
264 | t += sprintf(t, "W6692B_fill_fifo %c cnt %d", | 264 | t += sprintf(t, "W6692B_fill_fifo %c cnt %d", |
265 | bcs->channel + '1', count); | 265 | bcs->channel + '1', count); |
266 | QuickHex(t, ptr, count); | 266 | QuickHex(t, ptr, count); |
267 | debugl1(cs, bcs->blog); | 267 | debugl1(cs, "%s", bcs->blog); |
268 | } | 268 | } |
269 | } | 269 | } |
270 | 270 | ||