diff options
| author | Philipp Gesang <phg@phi-gamma.net> | 2018-11-07 23:40:26 +0100 |
|---|---|---|
| committer | Philipp Gesang <phg@phi-gamma.net> | 2018-11-07 23:40:32 +0100 |
| commit | 010d9d9d7f82e6d880da646c810492618476ee32 (patch) | |
| tree | 6f356707ca0019da3e27dc62377b379656be5c68 /misc | |
| parent | 39b9f77dbccecad50fb355cffb0e8e432e28f825 (diff) | |
| download | ocaml-sid-010d9d9d7f82e6d880da646c810492618476ee32.tar.gz | |
sid: sid_test: make subauthorities mandatory
Both the constructor “Sid.create” and the string format parser must
reject inputs lacking a subauthorities array of at least size one. Since
the array is no longer optional, reorder the the constructor arguments
to match the data representation.
It is still possible to create SIDs without subauthorities via the
“Sid.create_unsafe” constructor. Also, the packet representation will
happily accept them because their definition (as well as that that of
the identical RPC version) does not specify a minimum count.
This is all rather ambiguous and exacerbated by the fact that [MS-DTYP]
happily specifies an invalid SID “S-1-5” as the “NT_AUTHORITY”. However,
both the grammar and the Win API “ConvertStringSidToSidA()” function
reject SA-less inputs as invalid, so we should too.
Diffstat (limited to 'misc')
| -rw-r--r-- | misc/nix-ocaml-shell.nix | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/misc/nix-ocaml-shell.nix b/misc/nix-ocaml-shell.nix index 316f156..8a61a5b 100644 --- a/misc/nix-ocaml-shell.nix +++ b/misc/nix-ocaml-shell.nix @@ -3,6 +3,7 @@ with import <nixpkgs> {}; let libs = [ autoconf + gmp bubblewrap binutils curl |