www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 2767] New: DMD incorrectly mangles NTFS stream names

reply d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=2767

           Summary: DMD incorrectly mangles NTFS stream names
           Product: D
           Version: 2.026
          Platform: PC
        OS/Version: Windows
            Status: NEW
          Severity: trivial
          Priority: P5
         Component: DMD
        AssignedTo: bugzilla digitalmars.com
        ReportedBy: unknown simplemachines.org


Hardly a major issue, but on an NTFS formatted drive, try the following:

touch example:code.d
dmd example:code.d

DMD will generate a "code.exe" file, and use that as the module name.  While
again very trivial, this is incorrect - the filename is indeed
"example:code.d", it is an NTFS stream tied to the file named "example".

The usefulness of this bug is only on autogenerated files and make scripts,
which might create a file on Windows with a ":" in it, and then break in a hard
to track down way.  I found it only checking on another bug.

Minor patch upcoming.

-[Unknown]


-- 
Mar 28 2009
next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=2767


unknown simplemachines.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|bugzilla digitalmars.com    |unknown simplemachines.org
             Status|NEW                         |ASSIGNED




------- Comment #1 from unknown simplemachines.org  2009-03-28 23:05 -------
Created an attachment (id=295)
 --> (http://d.puremagic.com/issues/attachment.cgi?id=295&action=view)
Properly handle an NTFS stream filename.

This patch will consequently spit out an appropriate error unless the file has
a "module" line in it, since ":" is not a valid character in identifiers.

Comes free from correctly parsing in |FileName::name()|.

-[Unknown]


-- 
Mar 28 2009
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=2767





------- Comment #2 from braddr puremagic.com  2009-03-29 00:48 -------
Food for thought, I don't use windows much so my memory is foggy and quite
possibly out of date, but consider the old-school device names 'com1:'.  How,
if at all, do those interact with this code?

Also, your change is essentially a much more round about way of writing the
same code as was there before, no?

Before:
            case '\\':
            case ':':
                return e + 1;

After:
            case ':':
                if (e == str + 1)
                    return e + 1;
                // Intentional fall-through.
            case '\\':
                return e + 1;

Rearranged to duplicate the fallthrough block to better illustrate things:
  case ':':
    if (e == str + 1)
      return e + 1;
    return e + 1;
  case '\\':
    return e + 1;

So, case ':':
      return e + 1;
    case '\\':
      return e + 1;

And we're right back to the original code.

So, uh.. submit the wrong patch?  Did I misread something important and subtle
in the patch?


-- 
Mar 28 2009
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=2767





------- Comment #3 from bugzilla digitalmars.com  2009-03-29 01:35 -------
Brad makes a good point, there are aux:, lpt1:, prn:, com1:, etc. But they all
share the characteristic that the ':' is the last character. So, here's my
patch:

char *FileName::name(const char *str)
{
    char *e;
    size_t len = strlen(str);

    e = (char *)str + len;
    for (;;)
    {
        switch (*e)
        {
#if linux || __APPLE__ || __FreeBSD__
            case '/':
               return e + 1;
#endif
#if _WIN32
            case '/':
            case '\\':
                return e + 1;
            case ':':
                /* The ':' is a drive letter only if it is the second
                 * character or the last character,
                 * otherwise it is an ADS (Alternate Data Stream) separator.
                 * Consider ADS separators as part of the file name.
                 */
                if (e == str + 1 || e == str + len - 1)
                    return e + 1;
#endif
            default:
                if (e == str)
                    break;
                e--;
                continue;
        }
        return e;
    }
}


-- 
Mar 28 2009
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=2767


unknown simplemachines.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |unknown simplemachines.org




------- Comment #4 from unknown simplemachines.org  2009-03-29 01:46 -------
(In reply to comment #2)
 So, uh.. submit the wrong patch?  Did I misread something important and subtle
 in the patch?

Sorry, you're right. I screwed up this patch, my bad. I had to backtrack and make a copy of the original source tree after I started, so I ended up with the wrong code (and forgot to retest after making patch.) The way Walter ordered the code was what I had intended, heh. (In reply to comment #3)
 Brad makes a good point, there are aux:, lpt1:, prn:, com1:, etc. But they all
 share the characteristic that the ':' is the last character. So, here's my
 patch:

Yes, and this is a much better patch than mine. I see that you return empty for "prn:" etc., and that spits out an invalid filename error, which is probably the original intent of this code's structure. I didn't realize that. Thanks, -[Unknown] --
Mar 28 2009
prev sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=2767


bugzilla digitalmars.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED




------- Comment #5 from bugzilla digitalmars.com  2009-04-01 13:50 -------
Fixed DMD 1.042 and 2.027


-- 
Apr 01 2009