ChubbyAnt / sedutil

SEDutil: For Intel and AMD Ryzen Systems
https://sedutil.com
94 stars 22 forks source link

LockOnReset reporting bug in TCG Enterprise #53

Closed david-okamoto-wdc closed 1 year ago

david-okamoto-wdc commented 1 year ago

--listLockingRange(s) doesn't show LockOnReset correctly. Probably fixable via:

         const bool WriteLocked          = response.getUint8(5+4*7) != 0;
         // LockOnReset list has at least one element
         const bool LockOnReset          = response.tokenIs(5+4*8) == STARTLIST
-                                        && response.tokenIs(5+4*8+1) == DTA_TOKENID_UINT;
+                                        && response.tokenIs(5+4*8+1) == DTA_TOKENID_UINT
+                                        && response.getUint8(5+4*8+1) != 0;
         delete session;

                if (output_format == sedutilReadable) {
david-okamoto-wdc commented 1 year ago

Nevermind, I think I misunderstood this field. Now, I think if an array exists, it's set but if the array is empty, it's not set.

david-okamoto-wdc commented 1 year ago

Closing - not a bug.