Skip to contentSkip to conversation

IPv6: fix refcnt of struct ip6_flowlabel

PATCH In review Yan Zheng 3 participants 3 messages Oct 24, 2005 – Oct 25, 2005

Subsystems (4)
lore
Share & tools
lore

Yan Zheng

author
#0 lore raw

PATCH 1/1 IPv6: fix refcnt of struct ip6_flowlabel

net/ipv6/ip6_flowlabel.c +0 -0
Index: net/ipv6/ip6_flowlabel.c
===================================================================
@@ -483,7 +483,7 @@
						goto done;
					}
					fl1 = sfl->fl;
-					atomic_inc(&fl->users);
+					atomic_inc(&fl1->users);
					break;
				}
			}
#1 lore raw
Quoted 8 lines · Yan Zheng

Index: net/ipv6/ip6_flowlabel.c
===================================================================

net/ipv6/ip6_flowlabel.c 2005-10-24 19:55:23.000000000 +0800 line 487

Quoted 6 lines
net/ipv6/ip6_flowlabel.c 2005-10-24 19:55:23.000000000 +0800 renamed from net/ipv6/ip6_flowlabel.c 2005-10-22 10:31:13.000000000 +0800 +1 -1
@@ -483,7 +483,7 @@
                                                goto done;
                                        }
                                        fl1 = sfl->fl;
-                                       atomic_inc(&fl->users);
+                                       atomic_inc(&fl1->users);
                                        break;

Looks OK to me, Yoshifuji-san, ACK?

- Arnaldo

#2 lore raw

In article <[email protected]> (at Mon, 24 Oct 2005 16:35:33 -0200), Arnaldo Carvalho de Melo <[email protected]> says:

Quoted 17 lines · Yan Zheng
Quoted 8 lines

Index: net/ipv6/ip6_flowlabel.c
===================================================================

net/ipv6/ip6_flowlabel.c 2005-10-24 19:55:23.000000000 +0800 line 487

Quoted 6 lines
net/ipv6/ip6_flowlabel.c 2005-10-24 19:55:23.000000000 +0800 renamed from net/ipv6/ip6_flowlabel.c 2005-10-22 10:31:13.000000000 +0800 +1 -1
@@ -483,7 +483,7 @@
                                                goto done;
                                        }
                                        fl1 = sfl->fl;
-                                       atomic_inc(&fl->users);
+                                       atomic_inc(&fl1->users);
                                        break;

Looks OK to me, Yoshifuji-san, ACK?

--yoshfuji