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

Ability to get any permissions (including SYSDBA) working with services [CORE2657] #3063

Closed
firebird-automations opened this issue Oct 2, 2009 · 5 comments

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: @AlexPeshkoff

Using poor documented (at least not mentioned by Borland in 6.0 beta doc) tag isc_spb_command_line one can pass arbitrary trusted user name to service.

Commits: dbbea09

@firebird-automations
Copy link
Collaborator Author

Modified by: @AlexPeshkoff

assignee: Alexander Peshkov [ alexpeshkoff ]

@firebird-automations
Copy link
Collaborator Author

Commented by: @AlexPeshkoff

Fix contains 2 parts - first was needed to learn some utilities to accept minimum switch length in switch table. Next, service manager must detect that switches in command line.

I understand that in the worst case (when for example database name contains something like trusted_svc) error may be thrown in case of actually valid command line. Unfortunately, there is absolutely no way to distinguish between valid and wrong -trusted_role swicth in utility's command line parser.

The only correct solution is to stop passing normal parameters from services manager to services themself using command line, instead pass ready switchtable (which will be also passed from standalone utility main() function), but I wish to leave it for next FB version.

@firebird-automations
Copy link
Collaborator Author

Modified by: @AlexPeshkoff

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 2.5 RC1 [ 10362 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pcisar

status: Resolved [ 5 ] => Closed [ 6 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pavel-zotov

QA Status: No test

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

2 participants