linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* VFS bug? Trying to free free buffer
@ 2001-06-08 13:42 Alexey Vyskubov
  2001-06-10  7:09 ` Tachino Nobuhiro
  0 siblings, 1 reply; 2+ messages in thread
From: Alexey Vyskubov @ 2001-06-08 13:42 UTC (permalink / raw)
  To: linux-kernel

Hello.

Kernel 2.4.5.

$ sudo mount -o iocharset=garbage /dev/cdrom /cdrom

VFS: brelse: Trying to free free buffer

-- 
Alexey

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

* Re: VFS bug? Trying to free free buffer
  2001-06-08 13:42 VFS bug? Trying to free free buffer Alexey Vyskubov
@ 2001-06-10  7:09 ` Tachino Nobuhiro
  0 siblings, 0 replies; 2+ messages in thread
From: Tachino Nobuhiro @ 2001-06-10  7:09 UTC (permalink / raw)
  To: Alexey Vyskubov; +Cc: linux-kernel


Hello,

At Fri, 8 Jun 2001 16:42:54 +0300,
Alexey Vyskubov wrote:
> 
> Hello.
> 
> Kernel 2.4.5.
> 
> $ sudo mount -o iocharset=garbage /dev/cdrom /cdrom
> 
> VFS: brelse: Trying to free free buffer

  I think this is the ISOFS's bug. If invalid iocharset is specified
as a mount option, isofs_read_super() tries to free the buffer_head twice.

  This patch may help.



diff -r -u -N linux.org/fs/isofs/inode.c linux/fs/isofs/inode.c
--- linux.org/fs/isofs/inode.c	Sun Jun 10 14:23:02 2001
+++ linux/fs/isofs/inode.c	Sun Jun 10 15:10:38 2001
@@ -744,7 +744,7 @@
 		if (! s->u.isofs_sb.s_nls_iocharset) {
 			/* Fail only if explicit charset specified */
 			if (opt.iocharset)
-				goto out_freebh;
+				goto out_unlock;
 			s->u.isofs_sb.s_nls_iocharset = load_nls_default();
 		}
 	}


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

end of thread, other threads:[~2001-06-10  7:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2001-06-08 13:42 VFS bug? Trying to free free buffer Alexey Vyskubov
2001-06-10  7:09 ` Tachino Nobuhiro

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).