Re: [patch 3/5] Driver core: Documentation: use snprintf and strnlen

[Jan Veldeman]

Hi Ingo,

Ingo Oeser wrote:

> On Sunday 31 July 2005 13:12, domen@xxxxxxxxxxxx wrote:
> > From: Jan Veldeman <jan@xxxxxxx>
> > Documentation should give the good example of using snprintf and
> > strnlen in stead of sprintf and strlen.
> > 
> > PAGE_SIZE is used as the maximal length to reflect the behaviour of
> > show/store.
> 
> The whole part of the Documentation is obsoleted by the fact,
> that struct device has no structure member called "name".
> 
> People hacking sysfs should also try to hack the docu to match or
> at least remove the obsolete parts of it.
> 
> So you can drop this patch altogether, I think.
> 
> 

Even though the example doesn't work (the comment below the example even
indicates it shouldn't be done), I would suggest to still apply this
patch, as it shows the good usage of snprintf in stead of sprintf:

Somebody who looks at the documentation will certainly change the
contents of the attribute (this is by all means the reason for such an
example), but will not likely change the way sprintf is used.

Best regards,
Jan


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/