netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] net: cleanup miss usage of seq_release_private
@ 2004-07-07 19:26 OGAWA Hirofumi
  2004-07-07 21:10 ` David S. Miller
  0 siblings, 1 reply; 2+ messages in thread
From: OGAWA Hirofumi @ 2004-07-07 19:26 UTC (permalink / raw)
  To: David S. Miller; +Cc: netdev


These doesn't allocate memory and doesn't use seq->private. However
kfree() ignores NULL, so these are not the problem.

This patch just cleans these up.

Signed-off-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>
---

 net/ipv4/route.c              |    2 +-
 net/irda/discovery.c          |    2 +-
 net/irda/ircomm/ircomm_core.c |    2 +-
 net/irda/iriap.c              |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff -puN net/ipv4/route.c~net-seq_release-cleanup net/ipv4/route.c
--- linux-2.6.7/net/ipv4/route.c~net-seq_release-cleanup	2004-07-08 00:01:20.000000000 +0900
+++ linux-2.6.7-hirofumi/net/ipv4/route.c	2004-07-08 00:01:20.000000000 +0900
@@ -432,7 +432,7 @@ static struct file_operations rt_cpu_seq
 	.open	 = rt_cpu_seq_open,
 	.read	 = seq_read,
 	.llseek	 = seq_lseek,
-	.release = seq_release_private,
+	.release = seq_release,
 };
 
 #endif /* CONFIG_PROC_FS */
diff -puN net/irda/discovery.c~net-seq_release-cleanup net/irda/discovery.c
--- linux-2.6.7/net/irda/discovery.c~net-seq_release-cleanup	2004-07-08 00:01:20.000000000 +0900
+++ linux-2.6.7-hirofumi/net/irda/discovery.c	2004-07-08 00:01:20.000000000 +0900
@@ -449,6 +449,6 @@ struct file_operations discovery_seq_fop
 	.open           = discovery_seq_open,
 	.read           = seq_read,
 	.llseek         = seq_lseek,
-	.release	= seq_release_private,
+	.release	= seq_release,
 };
 #endif
diff -puN net/irda/iriap.c~net-seq_release-cleanup net/irda/iriap.c
--- linux-2.6.7/net/irda/iriap.c~net-seq_release-cleanup	2004-07-08 00:01:20.000000000 +0900
+++ linux-2.6.7-hirofumi/net/irda/iriap.c	2004-07-08 00:01:20.000000000 +0900
@@ -1093,7 +1093,7 @@ struct file_operations irias_seq_fops = 
 	.open           = irias_seq_open,
 	.read           = seq_read,
 	.llseek         = seq_lseek,
-	.release	= seq_release_private,
+	.release	= seq_release,
 };
 
 #endif /* PROC_FS */
diff -puN net/irda/ircomm/ircomm_core.c~net-seq_release-cleanup net/irda/ircomm/ircomm_core.c
--- linux-2.6.7/net/irda/ircomm/ircomm_core.c~net-seq_release-cleanup	2004-07-08 00:01:20.000000000 +0900
+++ linux-2.6.7-hirofumi/net/irda/ircomm/ircomm_core.c	2004-07-08 00:01:20.000000000 +0900
@@ -62,7 +62,7 @@ static struct file_operations ircomm_pro
 	.open           = ircomm_seq_open,
 	.read           = seq_read,
 	.llseek         = seq_lseek,
-	.release	= seq_release_private,
+	.release	= seq_release,
 };
 #endif /* CONFIG_PROC_FS */
 
_

-- 
OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] net: cleanup miss usage of seq_release_private
  2004-07-07 19:26 [PATCH] net: cleanup miss usage of seq_release_private OGAWA Hirofumi
@ 2004-07-07 21:10 ` David S. Miller
  0 siblings, 0 replies; 2+ messages in thread
From: David S. Miller @ 2004-07-07 21:10 UTC (permalink / raw)
  To: OGAWA Hirofumi; +Cc: netdev

On Thu, 08 Jul 2004 04:26:14 +0900
OGAWA Hirofumi <hirofumi@mail.parknet.co.jp> wrote:

> These doesn't allocate memory and doesn't use seq->private. However
> kfree() ignores NULL, so these are not the problem.
> 
> This patch just cleans these up.

Patch applied, thanks.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2004-07-07 21:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-07-07 19:26 [PATCH] net: cleanup miss usage of seq_release_private OGAWA Hirofumi
2004-07-07 21:10 ` David S. Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).