cvs commit: src/sys/geom/label g_label.c

[Pawel Jakub Dawidek]

On Thu, Aug 18, 2005 at 10:22:52PM -0700, Gordon Tetlow wrote:
+> >On Tue, Aug 16, 2005 at 05:18:49PM -0700, Doug Barton wrote:
+> >+> Pawel Jakub Dawidek wrote:
+> >+> +> >Because '/' creates a directory and I want each label to be represented
+> >+> >only by one file.
+> >+> +> I think what people are saying is that they like the directory creating behavior. Can you explain your rationale in more detail?
+> >Actually, I don't really care. All I wanted was one label to be represented
+> >by one single file. That's all. For me, leaving it as it is just asks for
+> >troubles.
+> >I can live without this change, really. This is something I'd like to ask
+> >about our TRB, but unfortunately it was retired yesterday:)
+> >CCing to freebsd-arch at .
+> >The question(s) is(are): Should we allow '/' in labels or should we replace
+> >it with something (eg. '_')? Maybe we should only deny labels with '/../'?
+> > 
+> When I wrote GEOM_VOL_FFS, I wrote it with the idea that you could make a heirarchy of providers in /dev/vol. Coming from an environment where it wasn't unusual for a 
+> single machine to have 30 to 40 disk available to it, it seemed natural that we should allow administrators the ability to define how they wanted things mapped out.
+> 
+> Now that I have just gone back and looked at the code that I wrote, I didn't allow non-alphanumerics in the volume name (although I actually didn't check it when creating 
+> the provider). I seem to recall making that decision specifically to get around the ../ tree traversal.
+> 
+> Anyway, I think it comes down to tools, not policy. I think "/" should be allowed.

Ok, guys, I backed-out the change.

-- 
Pawel Jakub Dawidek                       http://www.wheel.pl
pjd at FreeBSD.org                           http://www.FreeBSD.org
FreeBSD committer                         Am I Evil? Yes, I Am!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-src/attachments/20050820/d1d01a03/attachment.bin