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
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <469291264.232666.1632362629275.JavaMail.zimbra@sjtu.edu.cn>
Date:   Thu, 23 Sep 2021 10:03:49 +0800 (CST)
From:   郭志 <qtxuning1999@...u.edu.cn>
To:     "Marciniszyn, Mike" <mike.marciniszyn@...nelisnetworks.com>,
        "Dalessandro, Dennis" <dennis.dalessandro@...nelisnetworks.com>,
        dledford@...hat.com
Cc:     linux-rdma@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] infiniband hfi1: fix misuse of %x in ipoib_tx.c

I will change %p to %px and submit a new patch.

Thanks.

Guo

----- 原始邮件 -----
发件人: "Marciniszyn, Mike" <mike.marciniszyn@...nelisnetworks.com>
收件人: "Guo Zhi" <qtxuning1999@...u.edu.cn>, "Dalessandro, Dennis" <dennis.dalessandro@...nelisnetworks.com>, dledford@...hat.com
抄送: linux-rdma@...r.kernel.org, linux-kernel@...r.kernel.org
发送时间: 星期四, 2021年 9 月 23日 上午 1:51:08
主题: RE: [PATCH] infiniband hfi1: fix misuse of %x in ipoib_tx.c

> Subject: [PATCH] infiniband hfi1: fix misuse of %x in ipoib_tx.c
> 
> Pointers should be printed with %p or %px rather than cast to (unsigned long
> long) and printed with %llx.
> Change %llx to %p to print the pointer.
> 
> Signed-off-by: Guo Zhi <qtxuning1999@...u.edu.cn>

The unsigned long long was originally used to insure the entire accurate pointer as emitted.

This is to ensure the pointers in prints and event traces match values in stacks and register dumps.

I think the %p will obfuscate the pointer so %px is correct for our use case.

Mike

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ