* jamal <[EMAIL PROTECTED]> 2006-11-19 14:22 > On 11/19/06, jamal <[EMAIL PROTECTED]> wrote: > >This is probably 2.6.19 material (a little annoyance) > >I am testing using a new mailer - if this patch doesnt look right let me > know. > > Seems like i failed. Ok, heres the patch again..
HTML mail and application/octet-stream for the patch? :-) Probably never made it to netdev. The patch (enclosed again) is fine though. Dont add operations attribute headers when the family doesnt have any (gets annoying to see the header upon first registration before the commands are registered) signed-off-by: J Hadi Salim --- commit 0dc22b960a081832e6c05b59eeefbce564354b94 tree 0659cc668f40fbe383c8d63eaa3adbec72668244 parent e030f8294a5b9f8179dae10cdbf9dcf32aa64110 author J Hadi Salim [EMAIL PROTECTED] <[EMAIL PROTECTED]> Sun, 19 Nov 2006 14:00:04 -0500 committer J Hadi Salim [EMAIL PROTECTED] <[EMAIL PROTECTED](none)> Sun, 19 Nov 2006 14:00:04 -0500 net/netlink/genetlink.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/net/netlink/genetlink.c b/net/netlink/genetlink.c index 49bc2db..9e33d5f 100644 --- a/net/netlink/genetlink.c +++ b/net/netlink/genetlink.c @@ -403,6 +403,9 @@ static int ctrl_fill_info(struct genl_fa NLA_PUT_U32(skb, CTRL_ATTR_HDRSIZE, family->hdrsize); NLA_PUT_U32(skb, CTRL_ATTR_MAXATTR, family->maxattr); + if (list_empty(&family->ops_list)) + goto ctrl_fill_done; + nla_ops = nla_nest_start(skb, CTRL_ATTR_OPS); if (nla_ops == NULL) goto nla_put_failure; @@ -429,6 +432,7 @@ static int ctrl_fill_info(struct genl_fa nla_nest_end(skb, nest); } +ctrl_fill_done: nla_nest_end(skb, nla_ops); return genlmsg_end(skb, hdr); - 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