On Mon, 22 Aug 2011 08:33:49 -0500 Shirish Pargaonkar <shirishpargaonkar@xxxxxxxxx> wrote: > On Fri, Aug 12, 2011 at 11:33 AM, <shirishpargaonkar@xxxxxxxxx> wrote: > > From: Shirish Pargaonkar <shirishpargaonkar@xxxxxxxxx> > > > > > > Add mount option backup. > > > > It allows an authenticated user to access files with the intent to back them > > up including their ACLs, who may not have access permission but has > > "Backup files and directories user right" on them (by virtue of being part > > of the built-in group Backup Operators. > > > > If an authenticated user is not part of the built-in group Backup Operators > > at the server, access to such files is denied. > > > > > > Signed-off-by: Shirish Pargaonkar <shirishpargaonkar@xxxxxxxxx> > > --- > > > Jeff, Steve, > > Any comments on this patch (and manpage patch in cifs-utils)? > This seems like a really nasty kludge. It doesn't seem like the implications of this have been carefully considered. What happens I mount with the "backup" flag and do not have the necessary permissions on the server to use the flag in an open? Will this new flag be mutually exclusive with "multiuser"? One idea that might be better is to come up with way to mark certain (unix) users with the appropriate flag. If all the backup users were in a certan group, for instance, then you could use that info to decide whether to set the flag in the open calls. -- Jeff Layton <jlayton@xxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
