lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 21 Apr 2017 17:46:44 +0300 From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com> To: Andrew Morton <akpm@...ux-foundation.org>, linux-kernel@...r.kernel.org, Arnd Bergmann <arnd@...db.de>, Mika Westerberg <mika.westerberg@...ux.intel.com>, alsa-devel@...a-project.org, linux-input@...r.kernel.org, kvm@...r.kernel.org, devel@...uxdriverproject.org, linux-efi@...r.kernel.org, linux-acpi@...r.kernel.org Cc: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>, Matt Fleming <matt@...eblueprint.co.uk>, Ard Biesheuvel <ard.biesheuvel@...aro.org> Subject: [PATCH v1 7/8] efi: Use recently introduced uuid_le_cmp_p{p}() helpers Recently introduced helpers take pointers to uuid_{be|le} instead of reference. Using them makes code less ugly. Cc: Matt Fleming <matt@...eblueprint.co.uk> Cc: Ard Biesheuvel <ard.biesheuvel@...aro.org> Signed-off-by: Andy Shevchenko <andriy.shevchenko@...ux.intel.com> --- drivers/firmware/efi/cper.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/firmware/efi/cper.c b/drivers/firmware/efi/cper.c index d42537425438..7735b0f0ea90 100644 --- a/drivers/firmware/efi/cper.c +++ b/drivers/firmware/efi/cper.c @@ -402,14 +402,14 @@ static void cper_estatus_print_section( printk("%s""fru_text: %.20s\n", pfx, gdata->fru_text); snprintf(newpfx, sizeof(newpfx), "%s%s", pfx, INDENT_SP); - if (!uuid_le_cmp(*sec_type, CPER_SEC_PROC_GENERIC)) { + if (!uuid_le_cmp_p(sec_type, CPER_SEC_PROC_GENERIC)) { struct cper_sec_proc_generic *proc_err = (void *)(gdata + 1); printk("%s""section_type: general processor error\n", newpfx); if (gdata->error_data_length >= sizeof(*proc_err)) cper_print_proc_generic(newpfx, proc_err); else goto err_section_too_small; - } else if (!uuid_le_cmp(*sec_type, CPER_SEC_PLATFORM_MEM)) { + } else if (!uuid_le_cmp_p(sec_type, CPER_SEC_PLATFORM_MEM)) { struct cper_sec_mem_err *mem_err = (void *)(gdata + 1); printk("%s""section_type: memory error\n", newpfx); if (gdata->error_data_length >= @@ -418,7 +418,7 @@ static void cper_estatus_print_section( gdata->error_data_length); else goto err_section_too_small; - } else if (!uuid_le_cmp(*sec_type, CPER_SEC_PCIE)) { + } else if (!uuid_le_cmp_p(sec_type, CPER_SEC_PCIE)) { struct cper_sec_pcie *pcie = (void *)(gdata + 1); printk("%s""section_type: PCIe error\n", newpfx); if (gdata->error_data_length >= sizeof(*pcie)) -- 2.11.0
Powered by blists - more mailing lists