+[8.50b13] Bad work with user-restricted dirs via elevation

Bug reports will be moved here when the described bug has been fixed

Moderators: Hacker, petermad, Stefan2, white

Post Reply
User avatar
MVV
Power Member
Power Member
Posts: 8711
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

+[8.50b13] Bad work with user-restricted dirs via elevation

Post by *MVV »

I noticed it today in TCx32 8.50b13, and I can reproduce it with clean INI, and I can't reproduce it with previous betas or 8.01. Tested using Win7x64SP1.

Steps are:
1. Start TC with clean INI.
2. Enter C:\PerfLogs dir (or any other dir which is not allowed to access w/o elevation) and choose 'As Administrator' option. Moment X.
3. Copy any file from another panel to C:\PerfLogs, choose 'As Administrator'. Moment X.
4. Re-enter C:\PerfLogs some times. Moments X.


Moment X: TC may show empty folder with the only '..' item, or it may show empty panel (even w/o '..' item) with path c:\, or it may show wrong items (probably taken from different dir; '..' item may also be absent), or it may switch to TC dir, and it copes file to wrong dir too.


I think it may be related to switching to safe dir when TC loses focus.
User avatar
Horst.Epp
Power Member
Power Member
Posts: 7012
Joined: 2003-02-06, 17:36 UTC
Location: Germany

Post by *Horst.Epp »

I can confirm there is something strange.
For me it copies the file not into perflogs but in the root of C:
and also automatically changes into the root dir.
Sometimes the copy uses some complete unrelated dir and changes into it.
Tested with TC 8.50 b13 x86 and x64 version.
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 50865
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

Thanks, this seems to be related to moving many functions to background threads. The problem is that SetCurrentDirectory fails with these locations, because only the Administrator account can reach them. I will check how this error was handled in beta 12, and check whether it can be handled with multi-threading too.
Author of Total Commander
https://www.ghisler.com
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 50865
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

This should work again in beta 14, please try it!
Author of Total Commander
https://www.ghisler.com
User avatar
MVV
Power Member
Power Member
Posts: 8711
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

Post by *MVV »

Thanks, works in b14!
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 50865
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

Thanks!
Author of Total Commander
https://www.ghisler.com
Post Reply