Files icon indicating copy to clipboard operation
Files copied to clipboard

Bug: Failing to eject mounted iso

Open chkarsten opened this issue 3 years ago • 13 comments

What feature or improvement do you think would benefit Files?

Hi

I noticed that when mounting an .iso image, you cannot remove the iso image disc when clicking eject. There should be an option to remove the image mount disc permanently.

This is the only concern I have when working with iso images using files.

Steps to repro

  1. Right click iso to mount it
  2. Right click the mounted drive in the sidebar and click eject
  3. Observe that the drive isn't ejected

Files Version

4.5.21

Windows Version

Windows 11 22H2 22621.963

Comments

No response

chkarsten avatar Dec 15 '22 06:12 chkarsten

I suggest this should merge with the complete bug report: #11466

carlosvsilva avatar Feb 25 '23 16:02 carlosvsilva

I can reproduce this issue in v2.5.29

yaira2 avatar Oct 10 '23 15:10 yaira2

Hi. Still reproducable in version 3.0.15.0

mkikets99 avatar Dec 18 '23 19:12 mkikets99

Hi, I tried to reproduce it, but I don't see a mount option when right click an .iso file. image

SerhiiHarashchenko avatar Feb 23 '24 14:02 SerhiiHarashchenko

It will be in the show more options menu

Josh65-2201 avatar Feb 23 '24 14:02 Josh65-2201

Sorry, I can't still see it. I use the latest version from the source code. image

SerhiiHarashchenko avatar Feb 23 '24 14:02 SerhiiHarashchenko

@SerhiiHarashchenko

image

0x5bfa avatar Feb 26 '24 04:02 0x5bfa

I found that the flag IOCTL_STORAGE_EJECT_MEDIA we are using has been deprecated. Instead, should use CM_Request_Device_Eject.

0x5bfa avatar Feb 26 '24 05:02 0x5bfa

Hi all!

I'm new to this repo but would like to give this a go, where should I look in the repo for this issue?

I'm also able to build the project and reproduce this issue.

TcPirate1 avatar May 19 '24 21:05 TcPirate1

I guess this issue is too complex for new bee because this requires lower layer's win32 api

I was planning to fix this issue when merging all drive/device functions into IStorageRemovableDrivesService in #14572

0x5bfa avatar May 20 '24 02:05 0x5bfa

But you can find the problem code in EjectAsync in RemovableDevice in Files.App.Data.Models.

0x5bfa avatar May 20 '24 02:05 0x5bfa

I see, I did not realize it would be like that. I shall leave it in more experienced hands as I do not have experience with lower layer APIs.

Thanks for the response anyways!

TcPirate1 avatar May 20 '24 21:05 TcPirate1

@TcPirate1 When you have time you definitely can try out.

Fyi, we have project board, which enumerates all ready-to-build issue including bug fixes and feature requests. It also has size and priority of issues, enabling contributors to find easy-to-work issues. https://github.com/orgs/files-community/projects/3/views/7?sortedBy%5Bdirection%5D=desc&sortedBy%5BcolumnId%5D=13947857

0x5bfa avatar May 21 '24 13:05 0x5bfa

This should be resolved in the next update.

yaira2 avatar Sep 03 '24 01:09 yaira2