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

System Volume Information #151

Open
avogelba opened this issue Nov 16, 2016 · 8 comments
Open

System Volume Information #151

avogelba opened this issue Nov 16, 2016 · 8 comments

Comments

@avogelba
Copy link
Owner

Originally reported by: Anonymous


Hi,

I was wondering if you guys are going to add the ability for windirstat to view the System Volume Information folder in Windows.
Or is there a way to turn this on?

Thanks


@avogelba
Copy link
Owner Author

Original comment by gen btc (Bitbucket: genBTC, GitHub: genBTC):


Run As admin? Take ownership of the folder manually?

@avogelba
Copy link
Owner Author

Original comment by mirh_ (Bitbucket: mirh_, GitHub: Unknown):


In my opinion, the program should just be able to read the folder size, end.
Everything else inside it seems pretty.. Hazardous - which should only be touched with Windows own utilities.

Unfortunately, afaik folder size is always calculated by "iterating" size of single files inside.
..
It feels so strange though, for windows not to have an internal "size tracker" buried somewhere.

@avogelba
Copy link
Owner Author

Original comment by gen btc (Bitbucket: genBTC, GitHub: genBTC):


It has to do with following standards and obeying system security permissions. You would essentially be asking for the program to override NTFS security descriptors saying it shouldnt go in there. The best way to access that folder is what i said, and does work as shown: https://puu.sh/wPulz/00f3f3ce8b.png

This is a windows issue.

@avogelba
Copy link
Owner Author

Original comment by mirh_ (Bitbucket: mirh_, GitHub: Unknown):


Doing what you are saying is overriding security and "not following standards".

Besides, giving full control permissions is totally overkill for what the program needs to do.

@avogelba
Copy link
Owner Author

Original comment by gen btc (Bitbucket: genBTC, GitHub: genBTC):


Exactly. My point is that the burden of that should be on the user, not built into the program. I wasnt suggesting the program have full control.

Forgive me for trying to understand the argument, its almost as if you're suggesting that WindirStat is more special than every other program who can't get into that system directory because it should be allowed to know the size. And to override these security permissions to find out the size is, trivial, so it should just do it ? (assuming it could get it done with less than full control) ?

@avogelba
Copy link
Owner Author

Original comment by mirh_ (Bitbucket: mirh_, GitHub: Unknown):


Doing it with less than full control should be quite doable.
Reading should be enough.
Even better, with advanced permissions, only List Folder/Read Data should be required.

Then, to be really honest, above I was hoping for being able to query some whatever windows interface (that could possibly already keep count of it - e.g. disk cleanup has some kind of access to the SVI folder, if it can delete restore points).

But now that I looked a bit around.. Situation isn't so black.
At the very least you could make a "WinDirStat" security group with just the aforementioned permissions to have the program run if user request SVI info.

Otherwise, I read could do some tricks to flag itself as a backup application.
Last but not least, it could rely on launching another windows built-in program to retrieve the thing. Or magical robocoy.

Also, TIL SVI folder is not just about system restore.

@avogelba
Copy link
Owner Author

Original comment by gen btc (Bitbucket: genBTC, GitHub: genBTC):


Now that i put 2+2 together, this relates to this as well: https://bitbucket.org/windirstat/windirstat/issues/159/windirstat-not-finding-redis-data-files#comment-38257831

Claim the SeBackupPrivelege and act as a "backup application", I can agree with this.

@avogelba
Copy link
Owner Author

Original comment by mirh_ (Bitbucket: mirh_, GitHub: Unknown):


Oh. Cool that shooting into the pile solved something eventually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant