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

[Plus 1.1.0] Multiple bug reports and suggestions #1885

Closed
RandomGOTI opened this issue May 25, 2022 · 15 comments
Closed

[Plus 1.1.0] Multiple bug reports and suggestions #1885

RandomGOTI opened this issue May 25, 2022 · 15 comments
Labels
Status: Fixed in Next Build Fixed in the next Sandboxie version Type: Bug Something isn't working Type: Suggestions Suggestions to fix or introduce something Workaround Temporary or alternative solution

Comments

@RandomGOTI
Copy link

RandomGOTI commented May 25, 2022

  1. Using 'UseRegDeleteV2=y' option :
    Start.exe (4192): SBIE2306 Could not locate user directory: [C0000005 / 11]
    Start.exe (4192): SBIE2304 Initialization failed for process Start.exe
    Edit: without separate user folders :
    Start.exe (3912): SBIE2204 Cannot start sandboxed service RpcSs (C0000005)
    Start.exe (3912): SBIE2204 Cannot start sandboxed service DcomLaunch (-4)

  2. View -> Refresh View only works if 'WatchBoxSize=true'
    Steps to reproduce:

  • WatchBoxSize=false or missing
  • Start Run sandboxed dialog and close it so the status shows anything but empty
  • Manually delete the sandbox container via explorer
  • Refresh view won't update to show the status as empty unless the user reloads the ini

---Feature suggestion : add F5 keybind support for refresh view

  EDIT: UI lang setting to 'Auto Detection' was causing this issue not the box size info feature

3. "Count and display the disk space occupied by each sandbox" global option
  Will delete all the sandbox groups (BoxDisplayOrder)
  That is unnecessary even reseting the GUI

  Just adding 'WatchBoxSize=true' to Plus ini is enough and without losing customizations

  1. Application Error related to 'WatchBoxSize=true' after box session ended (all proccesses in a box are termianted) :

Faulting application name: SandMan.exe, version: 1.1.0.0
Faulting module name: ntdll.dll, version: 6.1.7601.23915
Exception code: 0xc0000374
Fault offset: 0x00000000000bf3e2
Faulting application path: C:\Program Files\Sandboxie\SandMan.exe
Faulting module path: C:\Windows\SYSTEM32\ntdll.dll

Steps to reproduce:

  • Start any program or Run sandboxed dialog and close it
  • Sandman takes a coffee break
    1
  • Application Error EventID 1000 in event viewer

*Sandman does not create a dmp with this crash , if this may help attached the dll
ntdll.zip

---Suggestion: Option that controls whenever or not the diskcache entry is updated automatically when a box session ends, would allow others that have the same issue to still make use of the box size column feature

  1. Using 'UseFileDeleteV2=y' option :
    Sanboxed explorer window won't show when first launched , requires to termiante process and relauch to have the window appear.
    Will freeze on shift delete action , requires to be terminated, the deleted file by the sandboxed explorer will be listed to the .dat file despite that it will still be visible on a relaunched sandboxed explorer

  2. 1.1.0 Recover Files Bug 2.0 #1840 (comment)

Windows 7 Ultimate SP1 x64
Plus 1.1.0 x64

@RandomGOTI RandomGOTI added the Confirmation Pending Further confirmation is requested label May 25, 2022
@droped
Copy link

droped commented May 25, 2022

I have the first issue too. It only happens with Separate User Folders on.

@DavidXanatos
Copy link
Member

I can only confirm the Separate User Folders at this moment, the other issues may be windows 7 releated will have to test it on that platform

Performance with this new in general is yet not fully optimized, so reports of how to reproduce bad performance are much appriciated to test it and fix it.

@DavidXanatos
Copy link
Member

I have tried the latest build on a windows 7 VM and it seams to work fine, except for the separate folder issue.
Also switching that option did not delete any box order informations.
Please try a clean install, or test on a clean VM

@RandomGOTI
Copy link
Author

RandomGOTI commented May 25, 2022

Also switching that option did not delete any box order informations. Please try a clean install, or test on a clean VM

happens with a new sandboxie-plus ini , opening the global options and clicking apply or ok will reset any customizations done beforehand
if instead of letting sandman create the ini have it with only this content the issue will not happen at all :
[Options]
UiLanguage=native

And as soon as i change from no translation to auto ,customizations are nuked again .. I had the wrong culprit

Correction on 2. Refresh View does nothing when "Count and display the disk spac..." option is off , with the option on it does update the status column

@isaak654 isaak654 changed the title [Plus 1.1.0] [Plus 1.1.0] Multiple bug reports to consider May 25, 2022
@isaak654 isaak654 changed the title [Plus 1.1.0] Multiple bug reports to consider [Plus 1.1.0] Multiple bug reports to fix May 25, 2022
@isaak654 isaak654 changed the title [Plus 1.1.0] Multiple bug reports to fix [Plus 1.1.0] Multiple bug reports and suggestions May 29, 2022
@isaak654
Copy link
Collaborator

For the record, the first post has been updated by the author for further reconsideration.

@DavidXanatos
Copy link
Member

Added

  • Refresh command is now bound to F5 #1885

Fixed

  • fixed issues with the new delete mechanism with "SeparateUserFolders=y" #1885
  • refresh now works without WatchBoxSize option #1885

so status:

  1. fixed

  2. fixed

  3. can't reproduce

  4. can't reproduce

  5. can't reproduce

  6. will look into it

@hg421
Copy link
Contributor

hg421 commented May 30, 2022

Just wanted to mention that I've also encountered issue 4 (the SandMan crash after closing the boxed process, when disk usage reporting is enabled), in Windows 10.

@isaak654
Copy link
Collaborator

isaak654 commented May 30, 2022

Just wanted to mention that I've also encountered issue 4 (the SandMan crash after closing the boxed process, when disk usage reporting is enabled), in Windows 10.

Confirmed. In order to reproduce the SandMan crash, I had to suddenly press "terminate all programs" while having a sandboxed MS Edge session with 4 tabs or more. The only difference with the error mentioned in OP is the absence of a message error, just a UI crash. I'm on the latest CI build v1.1.1 and disk usage option (WatchBoxSize=true) needs to be enabled in Sandboxie-Plus.ini before reproducing it.

@DavidXanatos
Copy link
Member

  1. fixed
  2. fixed
  3. can't reproduce
  4. fixed
  5. can't reproduce
  6. fixed

@DavidXanatos
Copy link
Member

So who else has issues 3 and 5 ?

@RandomGOTI
Copy link
Author

made a new issue for 3 since it's a wall of text on its own #1914

@isaak654 isaak654 added Type: Bug Something isn't working Type: Suggestions Suggestions to fix or introduce something and removed Confirmation Pending Further confirmation is requested labels Jun 4, 2022
@DavidXanatos
Copy link
Member

  1. fixed
  2. fixed
  3. fixed
  4. fixed
  5. can't reproduce
  6. fixed

@DavidXanatos DavidXanatos added the Status: Fixed in Next Build Fixed in the next Sandboxie version label Jun 4, 2022
@isaak654
Copy link
Collaborator

isaak654 commented Jun 4, 2022

I don't know if it's intended, but the F5 keybind for refresh view doesn't show any visual refresh when it's pressed on the latest CI build.

@isaak654 isaak654 added Confirmation Pending Further confirmation is requested and removed Status: Fixed in Next Build Fixed in the next Sandboxie version labels Jun 4, 2022
@DavidXanatos
Copy link
Member

on my test system it trigers a refresh, do you have 'WatchBoxSize=true' without that usually you wont see an afect of a refresh

@isaak654
Copy link
Collaborator

isaak654 commented Jun 5, 2022

on my test system it trigers a refresh, do you have 'WatchBoxSize=true' without that usually you wont see an afect of a refresh

Yes, it's correctly set in Sandboxie-Plus.ini. I have no clue why it doesn't work for me.

EDIT: I was able to make it work by resetting the GUI options:
Reset_GUI_options

@isaak654 isaak654 added Status: Fixed in Next Build Fixed in the next Sandboxie version Workaround Temporary or alternative solution and removed Confirmation Pending Further confirmation is requested labels Jun 5, 2022
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: Bug Something isn't working Type: Suggestions Suggestions to fix or introduce something Workaround Temporary or alternative solution
Projects
None yet
Development

No branches or pull requests

5 participants