Wei Yongjun 1bc4ee4088 sctp: fix warning at inet_sock_destruct() while release sctp socket ... Commit 'net: Move rx skb_orphan call to where needed' broken sctp protocol with warning at inet_sock_destruct(). Actually, sctp can do this right with sctp_sock_rfree_frag() and sctp_skb_set_owner_r_frag() pair. sctp_sock_rfree_frag(skb); sctp_skb_set_owner_r_frag(skb, newsk); This patch not revert the commit d55d87fdff, instead remove the sctp_sock_rfree_frag() function. ------------[ cut here ]------------ WARNING: at net/ipv4/af_inet.c:151 inet_sock_destruct+0xe0/0x142() Modules linked in: sctp ipv6 dm_mirror dm_region_hash dm_log dm_multipath scsi_mod ext3 jbd uhci_hcd ohci_hcd ehci_hcd [last unloaded: scsi_wait_scan] Pid: 1808, comm: sctp_test Not tainted 2.6.31-rc2 #40 Call Trace: [<c042dd06>] warn_slowpath_common+0x6a/0x81 [<c064a39a>] ? inet_sock_destruct+0xe0/0x142 [<c042dd2f>] warn_slowpath_null+0x12/0x15 [<c064a39a>] inet_sock_destruct+0xe0/0x142 [<c05fde44>] __sk_free+0x19/0xcc [<c05fdf50>] sk_free+0x18/0x1a [<ca0d14ad>] sctp_close+0x192/0x1a1 [sctp] [<c0649f7f>] inet_release+0x47/0x4d [<c05fba4d>] sock_release+0x19/0x5e [<c05fbab3>] sock_close+0x21/0x25 [<c049c31b>] __fput+0xde/0x189 [<c049c3de>] fput+0x18/0x1a [<c049988f>] filp_close+0x56/0x60 [<c042f422>] put_files_struct+0x5d/0xa1 [<c042f49f>] exit_files+0x39/0x3d [<c043086a>] do_exit+0x1a5/0x5dd [<c04a86c2>] ? d_kill+0x35/0x3b [<c0438fa4>] ? dequeue_signal+0xa6/0x115 [<c0430d05>] do_group_exit+0x63/0x8a [<c0439504>] get_signal_to_deliver+0x2e1/0x2f9 [<c0401d9e>] do_notify_resume+0x7c/0x6b5 [<c043f601>] ? autoremove_wake_function+0x0/0x34 [<c04a864e>] ? __d_free+0x3d/0x40 [<c04a867b>] ? d_free+0x2a/0x3c [<c049ba7e>] ? vfs_write+0x103/0x117 [<c05fc8fa>] ? sys_socketcall+0x178/0x182 [<c0402a56>] work_notifysig+0x13/0x19 ---[ end trace 9db92c463e789fba ]--- Signed-off-by: Wei Yongjun <yjwei@cn.fujitsu.com> Acked-by: Herbert Xu <herbert@gondor.apana.org.au> Acked-by: Vlad Yasevich <vladislav.yasevich@hp.com> Signed-off-by: David S. Miller <davem@davemloft.net>		2009-07-06 12:47:08 -07:00
..
associola.c	sctp: support non-blocking version of the new sctp_connectx() API	2009-06-03 09:14:47 -04:00
auth.c	fix similar typos to successfull	2009-01-08 08:31:15 -08:00
bind_addr.c	sctp: try harder to figure out address family when checking wildcards	2008-10-01 11:33:06 -04:00
chunk.c	SCTP: fix wrong debug counting of datamsg	2008-04-10 01:57:24 -07:00
command.c	[SCTP]: Remove sctp_add_cmd_sf wrapper bloat	2008-03-27 17:54:29 -07:00
debug.c	sctp: Add some missing types for debug message	2009-03-02 22:49:16 -08:00
endpointola.c	net: fix sctp breakage	2009-03-18 19:12:42 -07:00
input.c	sctp: Use frag list abstraction interfaces.	2009-06-09 00:24:07 -07:00
inqueue.c	[SCTP]: Stop claiming that this is a "reference implementation"	2008-02-05 10:59:07 -05:00
ipv6.c	ipv6: Use correct data types for ICMPv6 type and code	2009-06-23 04:31:07 -07:00
Kconfig	sctp: Don't abort initialization when CONFIG_PROC_FS=n	2008-07-18 23:03:44 -07:00
Makefile	sctp: Don't abort initialization when CONFIG_PROC_FS=n	2008-07-18 23:03:44 -07:00
objcnt.c	[NET]: Fix heavy stack usage in seq_file output routines.	2008-04-24 01:02:16 -07:00
output.c	sctp: xmit sctp packet always return no route error	2009-06-29 19:41:53 -07:00
outqueue.c	sctp: fix to send FORWARD-TSN chunk only if peer has such capable	2009-03-13 11:37:58 -07:00
primitive.c	[SCTP]: Stop claiming that this is a "reference implementation"	2008-02-05 10:59:07 -05:00
proc.c	sctp: remove sctp_assoc_proc_exit()	2008-07-22 14:21:30 -07:00
protocol.c	sctp: protocol.c call rcu_barrier() on unload.	2009-06-10 01:11:25 -07:00
sm_make_chunk.c	sctp: fix the error code when ASCONF is received with invalid address	2009-06-03 09:14:45 -04:00
sm_sideeffect.c	sctp: fix to choose alternate destination when retransmit ASCONF chunk	2009-06-03 09:14:46 -04:00
sm_statefuns.c	sctp: fix panic when T4-rto timer expire on removed transport	2009-06-03 09:14:46 -04:00
sm_statetable.c	sctp: fix a typo in net/sctp/sm_statetable.c	2009-06-03 09:14:45 -04:00
socket.c	sctp: fix warning at inet_sock_destruct() while release sctp socket	2009-07-06 12:47:08 -07:00
ssnmap.c	[SCTP]: Stop claiming that this is a "reference implementation"	2008-02-05 10:59:07 -05:00
sysctl.c	sctp: fix sack_timeout sysctl min and max types	2009-06-03 09:14:46 -04:00
transport.c	sctp: use time_before or time_after for comparing jiffies	2009-03-02 22:49:18 -08:00
tsnmap.c	trivial: fix then -> than typos in comments and documentation	2009-01-06 11:28:06 +01:00
ulpevent.c	sctp: Use frag list abstraction interfaces.	2009-06-09 00:24:07 -07:00
ulpqueue.c	net: Remove __skb_insert() calls outside of skbuff internals.	2008-09-21 21:28:51 -07:00