Bug 213 - Bug: System.Security.Principal.SecurityIdentifier.ParseSddlForm is not functioning properly
Summary: Bug: System.Security.Principal.SecurityIdentifier.ParseSddlForm is not functi...
Status: RESOLVED FEATURE
Alias: None
Product: Class Libraries
Classification: Mono
Component: mscorlib ()
Version: 2.10.x
Hardware: PC Windows
: --- minor
Target Milestone: Untriaged
Assignee: Bugzilla
URL:
Depends on:
Blocks:
 
Reported: 2011-08-06 16:17 UTC by Tal Aloni
Modified: 2011-08-07 23:05 UTC (History)
3 users (show)

Tags:
Is this bug a regression?: ---
Last known good build:


Attachments
Code snippet that demonstrates the problem (523 bytes, text/plain)
2011-08-06 16:17 UTC, Tal Aloni
Details


Notice (2018-05-24): bugzilla.xamarin.com is now in read-only mode.

Please join us on Visual Studio Developer Community and in the Xamarin and Mono organizations on GitHub to continue tracking issues. Bugzilla will remain available for reference in read-only mode. We will continue to work on open Bugzilla bugs, copy them to the new locations as needed for follow-up, and add the new items under Related Links.

Our sincere thanks to everyone who has contributed on this bug tracker over the years. Thanks also for your understanding as we make these adjustments and improvements for the future.


Please create a new report on GitHub or Developer Community with your current version information, steps to reproduce, and relevant error messages or log files if you are hitting an issue that looks similar to this resolved bug and you do not yet see a matching new report.

Related Links:
Status:
RESOLVED FEATURE

Description Tal Aloni 2011-08-06 16:17:34 UTC
Created attachment 76 [details]
Code snippet that demonstrates the problem

The attached code snippet demonstrate the problem,
the code works fine under MS Dot-Net Framework 2.0, and *should* work under Mono, but throws 'System.ArgumentException: Value was invalid.' when run under Mono 2.10.3

Please assist with the issue,

Regards,
Tal Aloni
Comment 1 Sebastien Pouliot 2011-08-07 10:54:37 UTC
The core of this code was contributed not so long ago. I suggest you email the author (or c.c. him on the bug report - if he's registered on bugzilla.xamarin.com) and ask him for help.

You could also contribute a fix yourself for the issue (and turn your sample into a unit test).

Downgrading priority since this is not used inside Mono itself (even on Windows) and won't work outside Windows.
Comment 2 Ken Bell 2011-08-07 15:44:13 UTC
I've posted a fix, and submitted a pull request for this and another bug I found.

https://github.com/mono/mono/pull/151

Note - the sample code still won't work, because Mono is unable to convert arbitrary account names to SID form.  Only well-known (built-in) Windows accounts can be converted by Mono.


Ken
Comment 3 Tal Aloni 2011-08-07 23:05:54 UTC
Thanks,
The important thing for now is to let the developer know what's not implemented via meaningful exceptions. now that I know what works, I can work around the problem.

Regards,
Tal Aloni