You are here

Home » Forums » Development Forums » Portable App Development

[Bug] PAL passing arguments to the main program when "RunAsAdmin" is set as "force" or "try"

11 posts / 0 new
Log in or register to post comments
Last post
December 19, 2012 - 1:48pm
#1
Jhonn
Jhonn's picture
Offline
Last seen: 1 week 9 hours ago
Joined: 2008-10-06 12:32
[Bug] PAL passing arguments to the main program when "RunAsAdmin" is set as "force" or "try"

I'm using the latest edition of PortableApps Launcher and found this bug when developing Dolphin Emulator Portable. When I set "RunAsAdmin" as try (needed in Dolphin), the main executable Dolphin.exe gets the argument "UAC" passed to the program executable, in case of Dolphin, making the emulator won't open, only showing "unknow command line argument" message...

The current workaround I'm using is calling the main executable through a BAT file, however this is weird and break passing of real CLI arguments through the launcher...

Top
December 19, 2012 - 1:52pm
John T. Haller
John T. Haller's picture
Online
Last seen: 23 sec ago
AdminDeveloperModeratorTranslator
Joined: 2005-11-28 22:21
Try

Why does it need try? Actually, why does it need admin at all?

Sometimes, the impossible can become possible, if you're awesome!

Top
December 19, 2012 - 2:02pm
(Reply to #2)
Jhonn
Jhonn's picture
Offline
Last seen: 1 week 9 hours ago
Joined: 2008-10-06 12:32
Need "try"

In computers running Windows XP and Vista, the XAudio2_7.dll isn't present, and this DLL in particular only works if I register it through regsvr32 before app run (of course, I unregister the DLL when the app close). Calling regsvr32 as a normal user make the registration fail with an 0x80070005 error (accordingly to Microsoft because the DLL need elevated privilegies), so, I ask to run as admin to make the DLL register work. If user doesn't have admin rights (or refuse the UAC prompt), the launcher warn the user that some features won't work, in this case, the Xaudio2 backend (needed in some games for proper audio emulation)...

Top
December 19, 2012 - 2:18pm
(Reply to #3)
John T. Haller
John T. Haller's picture
Online
Last seen: 23 sec ago
AdminDeveloperModeratorTranslator
Joined: 2005-11-28 22:21
Already Installed?

But if it is already installed, does it work without admin? If so, I'd suggest removing try and keeping a note about that. By policy, apps like games are not allowed to require or prompt for admin rights. Only system apps that would normally require admin. If certain functions of an app require admin in portable mode that the app itself does not require, it is unlikely that app will ever be released officially.

Sometimes, the impossible can become possible, if you're awesome!

Top
December 19, 2012 - 4:37pm
(Reply to #4)
Jhonn
Jhonn's picture
Offline
Last seen: 1 week 9 hours ago
Joined: 2008-10-06 12:32
Alright, but the bug exist!

In this case I'll remove the RunAsAdmin requirement from Dolphin Emulator Portable and then warn the user, however, the bug itself still here...

There is a fix or some workaround to this bug? Probably this is affecting all apps that use PAL, but I'm pretty sure that almost all of them doesn't care about unknown command line arguments, so, this bug wasn't discovered before.

Top
December 19, 2012 - 6:41pm
(Reply to #5)
John T. Haller
John T. Haller's picture
Online
Last seen: 23 sec ago
AdminDeveloperModeratorTranslator
Joined: 2005-11-28 22:21
Yup

Yes, this would still be a bug in PAL it seems when Try is set.

Sometimes, the impossible can become possible, if you're awesome!

Top
December 19, 2012 - 10:34pm
(Reply to #6)
Aluísio A. S. G.
Offline
Last seen: 7 years 2 months ago
DeveloperTranslator
Joined: 2010-11-09 17:43
Fixed 14 months ago

When PAL 3.0 was still called 2.2.

Previously known as kAlug.

Top
December 20, 2012 - 2:14pm
(Reply to #7)
Jhonn
Jhonn's picture
Offline
Last seen: 1 week 9 hours ago
Joined: 2008-10-06 12:32
Bug still here

I'm using PAL 2.2 (the latest version available here) and this bug isn't fixed, still present. If a fix was made it isn't present in the current version

Top
December 20, 2012 - 2:24pm
(Reply to #8)
Aluísio A. S. G.
Offline
Last seen: 7 years 2 months ago
DeveloperTranslator
Joined: 2010-11-09 17:43
Correct

It's fixed, just not released.

Previously known as kAlug.

Top
December 20, 2012 - 3:35pm
(Reply to #9)
Jhonn
Jhonn's picture
Offline
Last seen: 1 week 9 hours ago
Joined: 2008-10-06 12:32
Beta available?

Is there an beta release? If yes, where we can get it?

Top
March 3, 2014 - 3:49am
(Reply to #10)
Gord Caswell
Gord Caswell's picture
Offline
Last seen: 6 days 17 hours ago
DeveloperModerator
Joined: 2008-07-24 18:46
No beta, compile from source

There is no beta available, however you are able to compile from source. The source is available from SourceForge here: http://sourceforge.net/p/portableapps/launcher/ci/default/tree/

[edit] change download to compile

Top
Log in or register to post comments