Re: [PATCH 4/5] mountmgr.sys: Also unlink a:: drive when removing a dos device

[Alexandre Julliard]

Alexandre Goujon <ale.goujon(a)gmail.com> writes:
> @@ -818,6 +818,7 @@ NTSTATUS remove_dos_device( int letter, const char *udi )
>          if ((path = get_dosdevices_path( &p )))
>          {
>              p[0] = 'a' + drive->drive;
> +            unlink( path );

That's not done on purpose, to make it more likely that we use the same
drive letter next time the device is mounted.

-- 
Alexandre Julliard
julliard(a)winehq.org

[PATCH 4/5] mountmgr.sys: Also unlink a:: drive when removing a dos device

[GOUJON Alexandre]

On 08/31/2011 01:16 PM, Alexandre Julliard wrote:
> Alexandre Goujon<ale.goujon(a)gmail.com>  writes:
> > @@ -818,6 +818,7 @@ NTSTATUS remove_dos_device( int letter, const char *udi )
> >           if ((path = get_dosdevices_path(&p )))
> >           {
> >               p[0] = 'a' + drive->drive;
> > +            unlink( path );
> > That's not done on purpose, to make it more likely that we use the same
> > drive letter next time the device is mounted.

Sorry but your explanation doesn't convince me.

Here is a simple test case:
- open winecfg then the Drives tab
- make sure your fstab contains a mount point for a block device like 
/dev/cdrom
- click Autodetect then apply
- $ls -l .wine/dosdevices/
  you should see something like k: pointing to the mounted point like 
/media/cdrom et k:: pointing to /dev/cdrom with k the right letter
- remove k then apply
  k: has been removed but not k::

What happened : when you click on the apply button, 
apply_drive_changes() is called, then mountmgr_ioctl(), then 
define_unix_drive(), then remove_dos_device() which unlink() k: but not k::

[PATCH 4/5] mountmgr.sys: Also unlink a:: drive when removing a dos device

[Alexandre Julliard]

GOUJON Alexandre <ale.goujon(a)gmail.com> writes:
> What happened : when you click on the apply button,
> apply_drive_changes() is called, then mountmgr_ioctl(), then
> define_unix_drive(), then remove_dos_device() which unlink() k: but
> not k::

Like I said, that's on purpose.

-- 
Alexandre Julliard
julliard(a)winehq.org