Staging: otus: drop redundant memset
authorJulia Lawall <julia@diku.dk>
Tue, 9 Mar 2010 21:14:58 +0000 (22:14 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 11 May 2010 18:35:34 +0000 (11:35 -0700)
The region set by the call to memset is immediately overwritten by the
subsequent call to memcpy.

The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@@
expression e1,e2,e3,e4;
@@

- memset(e1,e2,e3);
  memcpy(e1,e4,e3);
// </smpl>

Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/otus/wwrap.c

index a74f7eea56e466e159f29700c716cede9a00e471..b02eb42cd796e2c6becf158e968454d7534e7f10 100644 (file)
@@ -956,7 +956,6 @@ int zfLnxCencSendMsg(struct sock *netlink_sk, u_int8_t *msg, int len)
        /*ÌîдÊý¾Ý±¨Ïà¹ØÐÅÏ¢*/
        nlh = NLMSG_PUT(skb, 0, 0, WAI_K_MSG, size-sizeof(*nlh));
        pos = NLMSG_DATA(nlh);
-       memset(pos, 0, len);
 
        /*´«Êäµ½Óû§¿Õ¼äµÄÊý¾Ý*/
        memcpy(pos, msg,  len);