xfrm: Use frag list abstraction interfaces.
David S. Miller [Tue, 9 Jun 2009 07:22:47 +0000 (00:22 -0700)]
Signed-off-by: David S. Miller <davem@davemloft.net>

net/xfrm/xfrm_algo.c

index 96036cf..d31ccb4 100644 (file)
@@ -696,8 +696,9 @@ int skb_icv_walk(const struct sk_buff *skb, struct hash_desc *desc,
 {
        int start = skb_headlen(skb);
        int i, copy = start - offset;
-       int err;
+       struct sk_buff *frag_iter;
        struct scatterlist sg;
+       int err;
 
        /* Checksum header. */
        if (copy > 0) {
@@ -742,28 +743,24 @@ int skb_icv_walk(const struct sk_buff *skb, struct hash_desc *desc,
                start = end;
        }
 
-       if (skb_shinfo(skb)->frag_list) {
-               struct sk_buff *list = skb_shinfo(skb)->frag_list;
-
-               for (; list; list = list->next) {
-                       int end;
-
-                       WARN_ON(start > offset + len);
-
-                       end = start + list->len;
-                       if ((copy = end - offset) > 0) {
-                               if (copy > len)
-                                       copy = len;
-                               err = skb_icv_walk(list, desc, offset-start,
-                                                  copy, icv_update);
-                               if (unlikely(err))
-                                       return err;
-                               if ((len -= copy) == 0)
-                                       return 0;
-                               offset += copy;
-                       }
-                       start = end;
+       skb_walk_frags(skb, frag_iter) {
+               int end;
+
+               WARN_ON(start > offset + len);
+
+               end = start + frag_iter->len;
+               if ((copy = end - offset) > 0) {
+                       if (copy > len)
+                               copy = len;
+                       err = skb_icv_walk(frag_iter, desc, offset-start,
+                                          copy, icv_update);
+                       if (unlikely(err))
+                               return err;
+                       if ((len -= copy) == 0)
+                               return 0;
+                       offset += copy;
                }
+               start = end;
        }
        BUG_ON(len);
        return 0;