From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <tim.lewis@insyde.com>
Received: from msmail.insydesw.com.tw (ms.insydesw.com [211.75.113.220])
 (using TLSv1 with cipher AES128-SHA (128/128 bits))
 (No client certificate requested)
 by ml01.01.org (Postfix) with ESMTPS id F01461A1E8D
 for <edk2-devel@lists.01.org>; Thu, 13 Oct 2016 11:16:10 -0700 (PDT)
Received: from msmail.insydesw.com.tw ([fe80::74f7:f173:f4aa:9a05]) by
 msmail.insydesw.com.tw ([fe80::74f7:f173:f4aa:9a05%11]) with mapi id
 14.01.0438.000; Fri, 14 Oct 2016 02:16:08 +0800
From: Tim Lewis <tim.lewis@insyde.com>
To: edk2-devel-01 <edk2-devel@lists.01.org>
Thread-Topic: [shell] Problems in EfiShellGetGuidFromName
Thread-Index: AdIlfdpU1v/3CG20S5OHaNM46qeDng==
Date: Thu, 13 Oct 2016 18:16:07 +0000
Message-ID: <7236196A5DF6C040855A6D96F556A53F3F660F@msmail.insydesw.com.tw>
Accept-Language: en-US, zh-TW
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
x-originating-ip: [192.168.100.108]
MIME-Version: 1.0
X-Content-Filtered-By: Mailman/MimeDel 2.1.21
Subject: [shell] Problems in EfiShellGetGuidFromName
X-BeenThere: edk2-devel@lists.01.org
X-Mailman-Version: 2.1.21
Precedence: list
List-Id: EDK II Development  <edk2-devel.lists.01.org>
List-Unsubscribe: <https://lists.01.org/mailman/options/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=unsubscribe>
List-Archive: <http://lists.01.org/pipermail/edk2-devel/>
List-Post: <mailto:edk2-devel@lists.01.org>
List-Help: <mailto:edk2-devel-request@lists.01.org?subject=help>
List-Subscribe: <https://lists.01.org/mailman/listinfo/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=subscribe>
X-List-Received-Date: Thu, 13 Oct 2016 18:16:11 -0000
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

In EfiShellGetGuidFromName (ShellProtocol.c), we see:

EfiShellGetGuidFromName(
  IN  CONST CHAR16   *GuidName,
  OUT       EFI_GUID *Guid
  )
{
  EFI_GUID    *NewGuid;
  EFI_STATUS  Status;

  if (Guid =3D=3D NULL || GuidName =3D=3D NULL) {
    return (EFI_INVALID_PARAMETER);
  }

  Status =3D GetGuidFromStringName(GuidName, NULL, &NewGuid);
  if (!EFI_ERROR(Status)) {
    CopyGuid(NewGuid, Guid);
  }
  return (Status);

However, this doesn't work. Notice that CopyGuid places the output paramete=
r as the 2nd parameter. But according to BaseMemoryLib:

GUID *
EFIAPI
CopyGuid (
  OUT GUID       *DestinationGuid,
  IN CONST GUID  *SourceGuid
  );

The destination should be the 1st parameter (not the second).