linux-ext4 - Re: [PATCH] fix check_mntent_file() to pass mode for open(O

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 9 Nov 2007 13:22:02 -0700
From:	Andreas Dilger <adilger@....com>
To:	Eric Sandeen <sandeen@...hat.com>
Cc:	Theodore Ts'o <tytso@....edu>, linux-ext4@...r.kernel.org
Subject: Re: [PATCH] fix check_mntent_file() to pass mode for open(O_CREAT)

On Nov 08, 2007  21:30 -0600, Eric Sandeen wrote:
> Andreas Dilger wrote:
> > On my FC8 install, ismounted.c fails to build because open(O_CREAT) is
> > used without passing a mode.  The following trivial patch fixes it.
> 
> You can add:
> 
> Acked-by: Eric Sandeen <sandeen@...hat.com>
> 
> 'cause it's an awful lot like the patch I sent for the same issue back
> on 8/16  ;-)  Guess I should have followed that up with a ping.  (though
> your 0600 mode is probably better than my 0644 was)

Oh, hmm.  I'm only testing against 1.40.2, so it is possible your fix got
in already...  Should have checked that first.

> Andreas, did you also run into trouble with struct_io_manager's ->open
> calls triggering this test?

No, didn't see any such problems with FC8-test3.

> I sent a patch for that,
> 
> [PATCH] rename ->open and ->close ops in struct_io_manager
> 
> too... maybe the glibc #define tricks got smarter and don't trigger that
> now?
> 
> -Eric
> 
> > Signed-off-by: Andreas Dilger <adilger@...ger.ca>
> > 
> > Index: e2fsprogs-1.40.2/lib/ext2fs/ismounted.c
> > ===================================================================
> > --- e2fsprogs-1.40.2.orig/lib/ext2fs/ismounted.c
> > +++ e2fsprogs-1.40.2/lib/ext2fs/ismounted.c
> > @@ -147,7 +147,7 @@ static errcode_t check_mntent_file(const
> >  is_root:
> >  #define TEST_FILE "/.ismount-test-file"		
> >  		*mount_flags |= EXT2_MF_ISROOT;
> > -		fd = open(TEST_FILE, O_RDWR|O_CREAT);
> > +		fd = open(TEST_FILE, O_RDWR|O_CREAT, 0600);
> >  		if (fd < 0) {
> >  			if (errno == EROFS)
> >  				*mount_flags |= EXT2_MF_READONLY;
> > 
> > Cheers, Andreas
> > --
> > Andreas Dilger
> > Sr. Software Engineer, Lustre Group
> > Sun Microsystems of Canada, Inc.
> > 
> > -
> > To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
> > the body of a message to majordomo@...r.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> -
> To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Cheers, Andreas
--
Andreas Dilger
Sr. Software Engineer, Lustre Group
Sun Microsystems of Canada, Inc.

-
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html