]> xenbits.xensource.com Git - people/royger/freebsd.git/commitdiff
libefivar: Fix byte orders of iSCSI.Lun
authorJose Luis Duran <jlduran@gmail.com>
Thu, 24 Feb 2022 01:55:30 +0000 (22:55 -0300)
committerWarner Losh <imp@FreeBSD.org>
Sun, 27 Feb 2022 16:12:47 +0000 (09:12 -0700)
Per UEFI spec, iSCSI.Lun is a 8-byte array with byte #0 in the left.
It means "0102030405060708" should be converted to:
    UINT8[8] = {01, 02, 03, 04, 05, 06, 07, 08}
or  UINT64 = {0807060504030201}

Today's implementation wrongly uses the reversed order.

Obtained from: https://github.com/tianocore/edk2/commit/d0196be1e39c419223738d7181d4a5d8972792d0
Pull Request:   https://github.com/freebsd/freebsd-src/pull/581

lib/libefivar/efivar-dp-parse.c

index c527607c71847e6dd12612bf3d33d9d13b0efee5..5d5d0dfb7baa73cb8fb8e10ec14220b6791db808 100644 (file)
@@ -2686,6 +2686,7 @@ DevPathFromTextiSCSI (
   CHAR16                      *ProtocolStr;
   CHAR8                       *AsciiStr;
   ISCSI_DEVICE_PATH_WITH_NAME *ISCSIDevPath;
+  UINT64                      Lun;
 
   NameStr           = GetNextParamStr (&TextDeviceNode);
   PortalGroupStr    = GetNextParamStr (&TextDeviceNode);
@@ -2704,7 +2705,8 @@ DevPathFromTextiSCSI (
   StrToAscii (NameStr, &AsciiStr);
 
   ISCSIDevPath->TargetPortalGroupTag = (UINT16) Strtoi (PortalGroupStr);
-  Strtoi64 (LunStr, &ISCSIDevPath->Lun);
+  Strtoi64 (LunStr, &Lun);
+  WriteUnaligned64 ((UINT64 *) &ISCSIDevPath->Lun, SwapBytes64 (Lun));
 
   Options = 0x0000;
   if (StrCmp (HeaderDigestStr, "CRC32C") == 0) {