Thread (25 messages) 25 messages, 9 authors, 2020-11-10

Re: [PATCH net-next 01/11] atm: horizon: shut up clang null pointer arithmetic warning

From: Xie He <hidden>
Date: 2020-10-27 04:02:21
Also in: lkml

On Mon, Oct 26, 2020 at 8:56 PM Xie He [off-list ref] wrote:
quoted
-  for (mem = (HDW *) memmap; mem < (HDW *) (memmap + 1); ++mem)
+  for (mem = (HDW *) memmap; mem < (HDW *) ((uintptr_t)memmap + 1); ++mem)
Note that these two lines are semantically different. In the first line,
"+ 1" moves the pointer by (sizeof memmap) bytes. However in the second
line, "+ 1" moves the pointer by only 1 byte.
Correction: in the first line "+ 1" moves the pointer by (sizeof *memmap) bytes.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help