Skip to content

Commit 9c5b3d8

Browse files
AhmedAhmed
authored andcommitted
Fix cross-platform strong name signing by manually parsing RSA key blobs
1 parent f8427ae commit 9c5b3d8

File tree

1 file changed

+20
-20
lines changed

1 file changed

+20
-20
lines changed

src/Compiler/AbstractIL/ilsign.fs

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// Copyright (c) Microsoft Corporation. All Rights Reserved. See License.txt in the project root for license information.
1+
// Copyright (c) Microsoft Corporation. All Rights Reserved. See License.txt in the project root for license information.
22

33
module internal FSharp.Compiler.AbstractIL.StrongNameSign
44

@@ -126,9 +126,7 @@ type BlobReader =
126126
val mutable _blob: byte array
127127
val mutable _offset: int
128128
new(blob: byte array) = { _blob = blob; _offset = 0 }
129-
130-
member x.Offset with get() = x._offset and set(v) = x._offset <- v
131-
129+
132130
member x.ReadInt32() : int =
133131
let offset = x._offset
134132
x._offset <- offset + 4
@@ -147,8 +145,7 @@ type BlobReader =
147145
let RSAParametersFromBlob blob keyType =
148146
let mutable reader = BlobReader blob
149147

150-
let header = reader.ReadInt32()
151-
if header <> 0x00000206 && header <> 0x00000207 && keyType = KeyType.KeyPair then
148+
if reader.ReadInt32() <> 0x00000207 && keyType = KeyType.KeyPair then
152149
raise (CryptographicException(getResourceString (FSComp.SR.ilSignPrivateKeyExpected ())))
153150

154151
reader.ReadInt32() |> ignore // ALG_ID
@@ -304,26 +301,25 @@ let signStream stream keyBlob =
304301
patchSignature stream peReader signature
305302

306303
let signatureSize (pk: byte array) =
307-
if pk.Length < 20 then 0
308-
else
309-
let reader = BlobReader pk
310-
reader.Offset <- 12
311-
let bitLen = reader.ReadInt32()
312-
let modulusLength = bitLen / 8
313-
314-
if modulusLength < 160 then 128 else modulusLength - 32
315-
// Key signing
316-
type keyContainerName = string
317-
type keyPair = byte array
318-
type pubkey = byte array
319-
type pubkeyOptions = byte array * bool
304+
let keySize = pk.Length
320305

306+
if keySize = 0 then 0
307+
else if keySize < 160 then 128
308+
else keySize - 32
309+
310+
// Returns a CLR Format Blob public key
321311
let getPublicKeyForKeyPair keyBlob =
322312
use rsa = RSA.Create()
323313
rsa.ImportParameters(RSAParametersFromBlob keyBlob KeyType.KeyPair)
324314
let rsaParameters = rsa.ExportParameters false
325315
toCLRKeyBlob rsaParameters CALG_RSA_KEYX
326316

317+
// Key signing
318+
type keyContainerName = string
319+
type keyPair = byte array
320+
type pubkey = byte array
321+
type pubkeyOptions = byte array * bool
322+
327323
let signerGetPublicKeyForKeyPair (kp: keyPair) : pubkey = getPublicKeyForKeyPair kp
328324

329325
let signerSignatureSize (pk: pubkey) : int = signatureSize pk
@@ -368,7 +364,11 @@ type ILStrongNameSigner =
368364

369365
member s.SignatureSize =
370366
let pkSignatureSize pk =
371-
signerSignatureSize pk
367+
try
368+
signerSignatureSize pk
369+
with exn ->
370+
failwith ("A call to StrongNameSignatureSize failed (" + exn.Message + ")")
371+
0x80
372372

373373
match s with
374374
| PublicKeySigner pk -> pkSignatureSize pk

0 commit comments

Comments
 (0)