dev_ioctl: use sizeof(x) instead of sizeof x

Also remove spaces after cast.

Signed-off-by: Fabian Frederick <fabf@skynet.be>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 6 insertions, 3 deletions

diff --git a/net/core/dev_ioctl.c b/net/core/dev_ioctl.c
index 72e899a3efda..b94b1d293506 100644
--- a/net/core/dev_ioctl.c
+++ b/net/core/dev_ioctl.c
@@ -142,10 +142,12 @@ static int dev_ifsioc_locked(struct net *net, struct ifreq *ifr, unsigned int cm
 
         case SIOCGIFHWADDR:
                 if (!dev->addr_len)
-                        memset(ifr->ifr_hwaddr.sa_data, 0, sizeof ifr->ifr_hwaddr.sa_data);
+                        memset(ifr->ifr_hwaddr.sa_data, 0,
+                               sizeof(ifr->ifr_hwaddr.sa_data));
                 else
                         memcpy(ifr->ifr_hwaddr.sa_data, dev->dev_addr,
-                               min(sizeof ifr->ifr_hwaddr.sa_data, (size_t) dev->addr_len));
+                               min(sizeof(ifr->ifr_hwaddr.sa_data),
+                                   (size_t)dev->addr_len));
                 ifr->ifr_hwaddr.sa_family = dev->type;
                 return 0;
 
@@ -265,7 +267,8 @@ static int dev_ifsioc(struct net *net, struct ifreq *ifr, unsigned int cmd)
                 if (ifr->ifr_hwaddr.sa_family != dev->type)
                         return -EINVAL;
                 memcpy(dev->broadcast, ifr->ifr_hwaddr.sa_data,
-                       min(sizeof ifr->ifr_hwaddr.sa_data, (size_t) dev->addr_len));
+                       min(sizeof(ifr->ifr_hwaddr.sa_data),
+                           (size_t)dev->addr_len));
                 call_netdevice_notifiers(NETDEV_CHANGEADDR, dev);
                 return 0;