On Sun, 2012-09-30 at 17:16 +0200, Johannes Berg wrote: > The internal function mac80211_format_buffer() has a > printf-style argument list, so add the attribute to > have gcc verify that list. [] > diff --git a/net/mac80211/debugfs.c b/net/mac80211/debugfs.c [] > @@ -17,8 +17,8 @@ [] > -int mac80211_format_buffer(char __user *userbuf, size_t count, > - loff_t *ppos, char *fmt, ...) > +int __printf(4, 5) mac80211_format_buffer(char __user *userbuf, size_t count, > + loff_t *ppos, char *fmt, ...) > { > va_list args; > char buf[DEBUGFS_FORMAT_BUFFER_SIZE]; Good idea, but this needs to be added to the prototype in debugfs.h not on the function itself. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html