crypto: Improve PER OID calculations
authorOndrej Holy <oholy@redhat.com>
Tue, 19 Dec 2017 13:42:06 +0000 (14:42 +0100)
committerOndrej Holy <oholy@redhat.com>
Tue, 19 Dec 2017 13:42:06 +0000 (14:42 +0100)
commit9f5d0d4c4d05020bec2201693e355c5010cfb097
tree79a7e7e70f55e7d3e3bf68c6ac5b9f89f1dca7c8
parent4791970c09b07d09cfa9ce7ff7899c1b47dc710d
crypto: Improve PER OID calculations

"(oid[0] << 4) & (oid[1] & 0x0F)" statement is always 0. It is not
problem currently because the only OID which is written by this
function should have 0 there. The function to read/write are pretty
limited anyway and can't work properly with all kind of OIDs. Maybe
it would be better to hardcode the OID there without decoding
and encoding. But those functions are already there so let's improve
them a bit according the spec and warn about limited set of
supported OIDs.

See:
https://msdn.microsoft.com/en-us/library/windows/desktop/bb540809
libfreerdp/crypto/per.c