Re: [Patch]: NUL and other special names

2004-06-03 Thread David Fritz
I don't how much you want to rely on undocumented features, but ntdll.dll exports a function called RtlIsDosDeviceName_U(). The WINE implementation has the following to say about it: /*** * RtlIsDosDeviceName_U (

Re: [Patch]: NUL and other special names

2004-06-03 Thread Pierre A. Humblet
David Fritz wrote: > > I don't how much you want to rely on undocumented features, but ntdll.dll > exports a function called RtlIsDosDeviceName_U(). The WINE implementation has > the following to say about it: > > /*** > *

Re: [Patch]: NUL and other special names

2004-06-03 Thread Igor Pechtchanski
On Thu, 3 Jun 2004, Pierre A. Humblet wrote: > David Fritz wrote: > > [snip] > > Also, from the patch: > > > > /* COM and LPT must be followed by a single digit */ > > > > The code in src/winsup/cygwin/devices.cc would seem to indicate that > > the number is not limited to a single digit

Re: [Patch]: NUL and other special names

2004-06-03 Thread Christopher Faylor
On Thu, Jun 03, 2004 at 03:53:40PM -0400, David Fritz wrote: >I don't how much you want to rely on undocumented features, but ntdll.dll >exports a function called RtlIsDosDeviceName_U(). The WINE implementation >has the following to say about it: > > >/***

Re: [Patch]: NUL and other special names

2004-06-03 Thread Pierre A. Humblet
Christopher Faylor wrote: > > On Thu, Jun 03, 2004 at 03:53:40PM -0400, David Fritz wrote: > >I don't how much you want to rely on undocumented features, but ntdll.dll > >exports a function called RtlIsDosDeviceName_U(). The WINE implementation > >has the following to say about it: > > > > > >/

Re: [Patch]: NUL and other special names

2004-06-03 Thread Pierre A. Humblet
Igor Pechtchanski wrote: > > On Thu, 3 Jun 2004, Pierre A. Humblet wrote: > > > David Fritz wrote: > > > > [snip] > > > Also, from the patch: > > > > > > /* COM and LPT must be followed by a single digit */ > > > > > > The code in src/winsup/cygwin/devices.cc would seem to indicate th

Re: [Patch]: NUL and other special names

2004-06-03 Thread David Fritz
Pierre A. Humblet wrote: [...] Does it handle conin$, conout$ and clocks$ ? It would appear that it does not (under Win2k SP4): 0x foo 0x0006 con 0x0006 nul 0x0006 prn 0x0008 lpt1 0x lpt16 0x0008 com1 0x com16 0x conin$ 0x conout

Re: [Patch]: NUL and other special names

2004-06-03 Thread Christopher Faylor
On Thu, Jun 03, 2004 at 04:55:05PM -0400, Pierre A. Humblet wrote: >>I believe that whenever I try to limit COM to single digits someone >>complains about their special board with 527 com ports or something. > >That's another issue. COM12 is not a DOS device (on NT), but it can be >the basename of

Re: [Patch]: NUL and other special names

2004-06-03 Thread Pierre A. Humblet
Christopher Faylor wrote: > > On Thu, Jun 03, 2004 at 04:55:05PM -0400, Pierre A. Humblet wrote: > >>I believe that whenever I try to limit COM to single digits someone > >>complains about their special board with 527 com ports or something. > > > >That's another issue. COM12 is not a DOS devic

Re: [Patch]: NUL and other special names

2004-06-03 Thread Pierre A. Humblet
"Pierre A. Humblet" wrote: > > Christopher Faylor wrote: > > > > On Thu, Jun 03, 2004 at 04:55:05PM -0400, Pierre A. Humblet wrote: > > >>I believe that whenever I try to limit COM to single digits someone > > >>complains about their special board with 527 com ports or something. > > > > > >That

Re: [Patch]: NUL and other special names

2004-06-03 Thread David Fritz
I don't think the logic implemented by RtlIsDosDeviceName_U() is all that different from the logic in the patch. Which is to say, it seems to use a hard-coded list of names and does not actually check for existing devices. Do we want to block all names that could be DOS devices or just the nam

Re: [Patch]: NUL and other special names

2004-06-03 Thread Pierre A. Humblet
David Fritz wrote: > > I don't think the logic implemented by RtlIsDosDeviceName_U() is all that > different from the logic in the patch. Which is to say, it seems to use a > hard-coded list of names and does not actually check for existing devices. Do > we want to block all names that could b

Re: [Patch]: NUL and other special names

2004-06-03 Thread Christopher Faylor
On Thu, Jun 03, 2004 at 05:26:45PM -0400, Pierre A. Humblet wrote: > > >Christopher Faylor wrote: >> >> On Thu, Jun 03, 2004 at 04:55:05PM -0400, Pierre A. Humblet wrote: >> >>I believe that whenever I try to limit COM to single digits someone >> >>complains about their special board with 527 com

Re: [Patch]: NUL and other special names

2004-06-03 Thread Christopher Faylor
On Thu, Jun 03, 2004 at 04:35:00PM -0400, Christopher Faylor wrote: >On Thu, Jun 03, 2004 at 03:53:40PM -0400, David Fritz wrote: >>I don't how much you want to rely on undocumented features, but ntdll.dll >>exports a function called RtlIsDosDeviceName_U(). The WINE implementation >>has the foll

Re: [Patch]: NUL and other special names

2004-06-03 Thread Christopher Faylor
On Thu, Jun 03, 2004 at 06:14:58PM -0400, Christopher Faylor wrote: >On Thu, Jun 03, 2004 at 04:35:00PM -0400, Christopher Faylor wrote: >>On Thu, Jun 03, 2004 at 03:53:40PM -0400, David Fritz wrote: >>>I don't how much you want to rely on undocumented features, but ntdll.dll >>>exports a function

XP crash (OT)

2004-06-03 Thread David Fritz
Christopher Faylor wrote: [...] Interestingly enough, I just added some checking to fhandler_base::open which used RtlIsDosDeviceName_U. It caused a reboot of my XP system every time I tried it. That's a first for XP. Oops. No, that was the result of passing garbage to InitializeObjectAttribute

Re: XP crash (OT)

2004-06-03 Thread Christopher Faylor
On Thu, Jun 03, 2004 at 07:11:20PM -0400, David Fritz wrote: >Christopher Faylor wrote: > >[...] >>>Interestingly enough, I just added some checking to fhandler_base::open >>>which >>>used RtlIsDosDeviceName_U. It caused a reboot of my XP system every time >>>I tried it. That's a first for XP. >

[Patch]: fchdir

2004-06-03 Thread Pierre A. Humblet
2004-06-04 Pierre Humblet <[EMAIL PROTECTED]> * path.cc (fchdir): Pass the Posix path to chdir. Index: path.cc === RCS file: /cvs/src/src/winsup/cygwin/path.cc,v retrieving revision 1.315 diff -u -p -r1.315 path.cc --- pat

Re: XP crash (OT)

2004-06-03 Thread David Fritz
Christopher Faylor wrote: On Thu, Jun 03, 2004 at 07:11:20PM -0400, David Fritz wrote: Christopher Faylor wrote: [...] Interestingly enough, I just added some checking to fhandler_base::open which used RtlIsDosDeviceName_U. It caused a reboot of my XP system every time I tried it. That's a first