walter harms <[EMAIL PROTECTED]> wrote: > These line >+ strcpy(bcbearer->media.name, "tipc-multicast"); > i gues that means tipc_bclink_name ?
The idea was to change how things are done, not _what_ is being done. > an even more secure version could be like this: > > > strncpy(bcbearer->media.name,sizeof(bcbearer->media.name),tipc_bclink_name); Ugh, please, no. The size of src is known in all cases; there is absoluty no point in using str(n|l)cpy here. > (in case someone ever changes the size of cbearer->media.name or > tipc_bclink_name and the hope > that wchat_t will never reach the kernel) In this case 'someone' should be really hurt, don't you think? Florian - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html