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

On Vintage View when the some menus are clicked the SandMan.exe crashes #2380

Closed
offhub opened this issue Oct 22, 2022 · 5 comments
Closed
Labels
crash dump Dump file attached for a detailed analysis fixed in next build Fixed in the next Sandboxie version

Comments

@offhub
Copy link
Collaborator

offhub commented Oct 22, 2022

Describe what you noticed and did

  1. Set the view as Vintage View
  2. Open the box settings
  3. Click on the General Options > Box Options or Restrictions
  4. SandMan.exe crashes.

How often did you encounter it so far?

Every time

Affected program

Not relevant

Download link

Not relevant

Where is the program located?

Not relevant to my request.

Expected behavior

It should work without crashing.

What is your Windows edition and version?

Windows 10 Pro 21H2 64-bit and Windows 11 Pro 21H2 64-bit

In which Windows account you have this problem?

User account with UAC protection set to Always notify.

Please mention any installed security software

Windows Defender

What version of Sandboxie are you running?

Sandboxie-Plus 1.4.2 and 1.5.0

Is it a new installation of Sandboxie?

I recently did a new clean installation.

Is it a regression?

No response

In which sandbox type you have this problem?

Not relevant to my request.

Can you reproduce this problem on an empty sandbox?

Not relevant to my request.

Did you previously enable some security policy settings outside Sandboxie?

No response

Crash dump

SandMan-v1.5.0 22.10.2022 18-33-21,168.dmp.zip

Trace log

No response

Sandboxie.ini configuration

No response

vintagecrash.mp4
@offhub offhub added the Confirmation pending Further confirmation is requested label Oct 22, 2022
@bastik-1001
Copy link
Contributor

bastik-1001 commented Oct 22, 2022

I can reproduce that with 1.5.0 (64bit) on Windows 7 (64bit), if the tree view is used and the vintage view. Vintage view and tabbed options interface do not trigger a crash.

Edit: And to be sure, advanced view with tree view options does not crash.

@okrc
Copy link
Contributor

okrc commented Oct 22, 2022

Already reproduced

The reason is that when iViewMode ! = 1, the advanced security tag is removed.

ui.tabsSecurity->removeTab(3); // advanced security

which raises an exception in ui.chkAddToJob->setChecked(bool).

if (GetAccessEntry(eWnd, "", eOpen, "*") != NULL)
{
ui.chkAddToJob->setEnabled(false);
ui.chkAddToJob->setChecked(false);
}
else
{
ui.chkAddToJob->setEnabled(true);
ui.chkAddToJob->setChecked(!m_pBox->GetBool("NoAddProcessToJob", false));
}

@bastik-1001
Copy link
Contributor

@okrc Are there ways to find these flaws by some sort of automatic analysis?

Unless I had toyed around with Sandboxie's display options, I never had produced the issue myself, since that isn't how I am using Sandboxie. That's why I am asking if there is something that can detect the broken logic of the code.

@okrc
Copy link
Contributor

okrc commented Oct 22, 2022

@bastik-1001 I'm sorry, I don't know if there is a better tool than GUI Testing.

@isaak654 isaak654 added the crash dump Dump file attached for a detailed analysis label Oct 22, 2022
@APMichael
Copy link
Contributor

JFYI: For me it also crashes when adding a new entry under "Resource Access", as soon as "Apply" is clicked.

@DavidXanatos DavidXanatos added the fixed in next build Fixed in the next Sandboxie version label Oct 23, 2022
@offhub offhub removed the Confirmation pending Further confirmation is requested label Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crash dump Dump file attached for a detailed analysis fixed in next build Fixed in the next Sandboxie version
Projects
None yet
Development

No branches or pull requests

6 participants