| Message ID | 20200514123729.156283-10-frankja@linux.ibm.com |
|---|---|
| State | New |
| Headers | show |
| Series | pc-bios: s390x: Cleanup part 1 | expand |
On 14/05/2020 14.37, Janosch Frank wrote: > Let's make it a bit more clear that we check the full 64 bits to fit > into the 32 we return. > > Signed-off-by: Janosch Frank <frankja@linux.ibm.com> > Suggested-by: David Hildenbrand <david@redhat.com> > Reviewed-by: David Hildenbrand <david@redhat.com> > --- > pc-bios/s390-ccw/helper.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/pc-bios/s390-ccw/helper.h b/pc-bios/s390-ccw/helper.h > index 78d5bc7442..8a50ec91bb 100644 > --- a/pc-bios/s390-ccw/helper.h > +++ b/pc-bios/s390-ccw/helper.h > @@ -18,7 +18,7 @@ > /* Avoids compiler warnings when casting a pointer to a u32 */ > static inline uint32_t ptr2u32(void *ptr) > { > - IPL_assert((uint64_t)ptr <= 0xffffffff, "ptr2u32: ptr too large"); > + IPL_assert((uint64_t)ptr <= 0xffffffffull, "ptr2u32: ptr too large"); > return (uint32_t)(uint64_t)ptr; > } > > Reviewed-by: Thomas Huth <thuth@redhat.com>
diff --git a/pc-bios/s390-ccw/helper.h b/pc-bios/s390-ccw/helper.h index 78d5bc7442..8a50ec91bb 100644 --- a/pc-bios/s390-ccw/helper.h +++ b/pc-bios/s390-ccw/helper.h @@ -18,7 +18,7 @@ /* Avoids compiler warnings when casting a pointer to a u32 */ static inline uint32_t ptr2u32(void *ptr) { - IPL_assert((uint64_t)ptr <= 0xffffffff, "ptr2u32: ptr too large"); + IPL_assert((uint64_t)ptr <= 0xffffffffull, "ptr2u32: ptr too large"); return (uint32_t)(uint64_t)ptr; }