The patch is good to me. Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-----Original Message----- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Wednesday, January 6, 2016 4:30 AM To: edk2-de...@ml01.01.org Cc: Subramanian Sriram <srira...@hpe.com>; El-Haj-Mahmoud Samer <samer.el-haj-mahm...@hpe.com>; Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Wu, Jiaxin <jiaxin...@intel.com> Subject: [PATCH] NetworkPkg: DnsDxe: fix return type of DnsFillinQNameForQueryIp() Change the return type of DnsFillinQNameForQueryIp() from (UINT8*) to (CHAR*). This brings the function in sync with both its internal variables and all of its call sites, fixing the following gcc build breakage: > NetworkPkg/DnsDxe/DnsImpl.c: In function 'DnsFillinQNameForQueryIp': > NetworkPkg/DnsDxe/DnsImpl.c:1068:3: error: pointer targets in return > differ in signedness > [-Werror=pointer-sign] > return QueryName; > ^ The code was added in git commit fcae1a99 (SVN r19579). Cc: Subramanian Sriram <srira...@hpe.com> Cc: El-Haj-Mahmoud Samer <samer.el-haj-mahm...@hpe.com> Cc: Ye Ting <ting...@intel.com> Cc: Fu Siyuan <siyuan...@intel.com> Cc: Jiaxin Wu <jiaxin...@intel.com> Cc: Ye Ting <ting...@intel.com> Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Laszlo Ersek <ler...@redhat.com> --- NetworkPkg/DnsDxe/DnsImpl.h | 2 +- NetworkPkg/DnsDxe/DnsImpl.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/NetworkPkg/DnsDxe/DnsImpl.h b/NetworkPkg/DnsDxe/DnsImpl.h index a13355f..847cd15 100644 --- a/NetworkPkg/DnsDxe/DnsImpl.h +++ b/NetworkPkg/DnsDxe/DnsImpl.h @@ -581,7 +581,7 @@ AddDns6ServerIp ( @return QName filled successfully. **/ -UINT8 * +CHAR8 * EFIAPI DnsFillinQNameForQueryIp ( IN CHAR16 *HostName diff --git a/NetworkPkg/DnsDxe/DnsImpl.c b/NetworkPkg/DnsDxe/DnsImpl.c index 8725670..362af86 100644 --- a/NetworkPkg/DnsDxe/DnsImpl.c +++ b/NetworkPkg/DnsDxe/DnsImpl.c @@ -1026,7 +1026,7 @@ AddDns6ServerIp ( @return QName filled successfully. **/ -UINT8 * +CHAR8 * EFIAPI DnsFillinQNameForQueryIp ( IN CHAR16 *HostName -- 1.8.3.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel