batman-adv: Calculate sizeof using variable insead of types

Documentation/CodingStyle recommends to use the form

	p = kmalloc(sizeof(*p), ...);

to calculate the size of a struct and not the version where the struct
name is spelled out to prevent bugs when the type of p changes. This
also seems appropriate for manipulation of buffers when they are
directly associated with p.

Signed-off-by: Sven Eckelmann <sven@narfation.org>

1 files changed, 10 insertions, 11 deletions

diff --git a/net/batman-adv/unicast.c b/net/batman-adv/unicast.c
index 82717fd9661d..6eabf42f8822 100644
--- a/net/batman-adv/unicast.c
+++ b/net/batman-adv/unicast.c
@@ -39,8 +39,8 @@ static struct sk_buff *frag_merge_packet(struct list_head *head,
                 (struct unicast_frag_packet *)skb->data;
         struct sk_buff *tmp_skb;
         struct unicast_packet *unicast_packet;
-        int hdr_len = sizeof(struct unicast_packet);
-        int uni_diff = sizeof(struct unicast_frag_packet) - hdr_len;
+        int hdr_len = sizeof(*unicast_packet);
+        int uni_diff = sizeof(*up) - hdr_len;
 
         /* set skb to the first part and tmp_skb to the second part */
         if (up->flags & UNI_FRAG_HEAD) {
@@ -53,7 +53,7 @@ static struct sk_buff *frag_merge_packet(struct list_head *head,
         if (skb_linearize(skb) < 0 || skb_linearize(tmp_skb) < 0)
                 goto err;
 
-        skb_pull(tmp_skb, sizeof(struct unicast_frag_packet));
+        skb_pull(tmp_skb, sizeof(*up));
         if (pskb_expand_head(skb, 0, tmp_skb->len, GFP_ATOMIC) < 0)
                 goto err;
 
@@ -99,8 +99,7 @@ static int frag_create_buffer(struct list_head *head)
         struct frag_packet_list_entry *tfp;
 
         for (i = 0; i < FRAG_BUFFER_SIZE; i++) {
-                tfp = kmalloc(sizeof(struct frag_packet_list_entry),
-                        GFP_ATOMIC);
+                tfp = kmalloc(sizeof(*tfp), GFP_ATOMIC);
                 if (!tfp) {
                         frag_list_free(head);
                         return -ENOMEM;
@@ -224,8 +223,8 @@ int frag_send_skb(struct sk_buff *skb, struct bat_priv *bat_priv,
         struct hard_iface *primary_if;
         struct sk_buff *frag_skb;
         struct unicast_frag_packet *frag1, *frag2;
-        int uc_hdr_len = sizeof(struct unicast_packet);
-        int ucf_hdr_len = sizeof(struct unicast_frag_packet);
+        int uc_hdr_len = sizeof(*unicast_packet);
+        int ucf_hdr_len = sizeof(*frag1);
         int data_len = skb->len - uc_hdr_len;
         int large_tail = 0, ret = NET_RX_DROP;
         uint16_t seqno;
@@ -250,14 +249,14 @@ int frag_send_skb(struct sk_buff *skb, struct bat_priv *bat_priv,
         frag1 = (struct unicast_frag_packet *)skb->data;
         frag2 = (struct unicast_frag_packet *)frag_skb->data;
 
-        memcpy(frag1, &tmp_uc, sizeof(struct unicast_packet));
+        memcpy(frag1, &tmp_uc, sizeof(tmp_uc));
 
         frag1->ttl--;
         frag1->version = COMPAT_VERSION;
         frag1->packet_type = BAT_UNICAST_FRAG;
 
         memcpy(frag1->orig, primary_if->net_dev->dev_addr, ETH_ALEN);
-        memcpy(frag2, frag1, sizeof(struct unicast_frag_packet));
+        memcpy(frag2, frag1, sizeof(*frag2));
 
         if (data_len & 1)
                 large_tail = UNI_FRAG_LARGETAIL;
@@ -314,7 +313,7 @@ find_router:
         if (!neigh_node)
                 goto out;
 
-        if (my_skb_head_push(skb, sizeof(struct unicast_packet)) < 0)
+        if (my_skb_head_push(skb, sizeof(*unicast_packet)) < 0)
                 goto out;
 
         unicast_packet = (struct unicast_packet *)skb->data;
@@ -328,7 +327,7 @@ find_router:
         memcpy(unicast_packet->dest, orig_node->orig, ETH_ALEN);
 
         if (atomic_read(&bat_priv->fragmentation) &&
-            data_len + sizeof(struct unicast_packet) >
+            data_len + sizeof(*unicast_packet) >
                                 neigh_node->if_incoming->net_dev->mtu) {
                 /* send frag skb decreases ttl */
                 unicast_packet->ttl++;