Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade from 5.33, OpenFilePath=* not work any more? #971

Closed
asbai opened this issue Jul 7, 2021 · 14 comments
Closed

Upgrade from 5.33, OpenFilePath=* not work any more? #971

asbai opened this issue Jul 7, 2021 · 14 comments
Labels
Status: Fixed in Next Build Fixed in the next Sandboxie version Type: Regression A Sandboxie build broke compatibility, it was working before Workaround Temporary or alternative solution

Comments

@asbai
Copy link

asbai commented Jul 7, 2021

I just upgraded from 5.33 to 5.50.2, then I found that the new version no longer correctly recognizes the OpenFilePath=* and OpenPipePath=* instructions.

I tried to modify them to OpenFilePath=*.* and OpenPipePath=*.* The problem was alleviated.

But when I tried to use WinRAR to perform decompression in a sandbox configured with OpenFilePath=*.* and OpenPipePath=*.*, these configurations still did not take effect (the decompressed files were thrown into the sandbox) .

How can I keep the same behavior as OpenFilePath=* and OpenPipePath=* in version 5.33?

Thanks :-)

@asbai
Copy link
Author

asbai commented Jul 7, 2021

I found OpenFilePath=*:\* do the trick.

@asbai asbai closed this as completed Jul 7, 2021
@isaak654 isaak654 added the Workaround Temporary or alternative solution label Jul 7, 2021
@isaak654
Copy link
Collaborator

isaak654 commented Sep 13, 2021

I think this regression started on v0.8.0 / 5.50.0.

It's fixed for me on v0.9.6 / 5.51.6, see also #1175 (reply in thread).

@isaak654 isaak654 added Status: Fixed in Next Build Fixed in the next Sandboxie version Type: Regression A Sandboxie build broke compatibility, it was working before labels Sep 13, 2021
@asbai
Copy link
Author

asbai commented Jan 5, 2022

This bug reproduced on 5.55.6, Now neither * nor *:\* will work anymore. :-(

@asbai asbai reopened this Jan 5, 2022
@isaak654
Copy link
Collaborator

isaak654 commented Jan 5, 2022

If it didn't occur with Classic 5.55.5, then there is a chance this will be fixed on next v5.55.7:

fixed issue with file renaming in open paths introduced in 1.0.6

@asbai
Copy link
Author

asbai commented Jan 5, 2022

We can also reproduce it on 5.55.5, version 5.53.3 is fine.

@isaak654
Copy link
Collaborator

isaak654 commented Jan 5, 2022

We can also reproduce it on 5.55.5, version 5.53.3 is fine.

In this case, please report which is the exact release that introduced the regression (including each pre-release in the middle).

@asbai
Copy link
Author

asbai commented Jan 5, 2022

In this case, please report which is the exact release that introduced the regression (including each pre-release in the middle).

The first version to reproduce this problem is 5.55.0

@DavidXanatos DavidXanatos added to investigate Priority: High To be done as soon as possible and removed Status: Fixed in Next Build Fixed in the next Sandboxie version labels Jan 5, 2022
@isaak654 isaak654 removed the Workaround Temporary or alternative solution label Jan 5, 2022
@DavidXanatos
Copy link
Member

What you need is
OpenFilePath=*
OpenPipePath=*

. does work for files but not directories
and *:* seams also wrong as internally nt paths are used which don't have : and while C:\ can be converted to an nt path if the volume is present *:\ can not

I tested with
OpenFilePath=*
OpenPipePath=*
and it worked as expected i.e. all paths were open except those explicitly not

@DavidXanatos DavidXanatos added Issue: Can't Reproduce The issue cannot be reproduced internally and removed to investigate Priority: High To be done as soon as possible labels Jan 6, 2022
@DavidXanatos
Copy link
Member

PS: you may also run into issues when you have folders in the sandbox in the location of open paths, so try clearing the sandbox if the issue persists

@isaak654 isaak654 added the Workaround Temporary or alternative solution label Jan 6, 2022
@asbai
Copy link
Author

asbai commented Jan 6, 2022

We tried:

  1. Deleting the drive directory in the sandbox directory;
  2. Starting notepad in the sandbox and modifying C:\Program Files\test-folder\test.txt.
  3. Starting from version 5.55.0, the drive directory will appear in the sandbox directory at this time, and the modified content will exist under drive\C\Program Files\test-folder\test.txt.

We tried * as well as *:\* and both behaved like this.

@asbai
Copy link
Author

asbai commented Jan 6, 2022

and : seams also wrong as internally nt paths are used which don't have : and while C:\ can be converted to an nt path if the volume is present *:\ can not

Let's tidy up the timeline:

  1. OpenFilePath=* worked fine in older versions (eg: 5.33) until 5.50.2.
  2. In version 5.50.2, OpenFilePath=*:\* must be used, OpenFilePath=* will not work.
  3. As of version 5.55.0, neither OpenFilePath=*:\* nor OpenFilePath=* work.

So OpenFilePath=*:\* should be no NT Kernel path matching problem.

Also, maybe in some version between 5.50.2 and 5.55.0, OpenFilePath=* used to work correctly, but we haven't tested it.

Thanks :-)

@DavidXanatos
Copy link
Member

I see... with the last verbose issue report I coudl replicate the issue,

It only apples to the following locations:
\Device\HarddiskVolume4\Program Files (x86)*
\Device\HarddiskVolume4\Program Files*
\Device\HarddiskVolume4\Windows*
other paths should work properly

a fix for that will be included in the built 1.0.8

@DavidXanatos DavidXanatos added Status: Fixed in Next Build Fixed in the next Sandboxie version and removed Issue: Can't Reproduce The issue cannot be reproduced internally labels Jan 6, 2022
@DavidXanatos
Copy link
Member

PS:
if you specify more specific paths like C\Program Files\test-folder this wil overrule the mentioned location as well and allow open access

@asbai
Copy link
Author

asbai commented Jan 6, 2022

OK, Thanks :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Fixed in Next Build Fixed in the next Sandboxie version Type: Regression A Sandboxie build broke compatibility, it was working before Workaround Temporary or alternative solution
Projects
None yet
Development

No branches or pull requests

3 participants