[PATCH 1/5] xorg: Fix a bunch of format string size warnings in debug code.

Keith Packard keithp at keithp.com
Mon May 12 16:21:58 PDT 2014


walter harms <wharms at bfs.de> writes:


>> -    DebugF("xf86ReadBIOS: BIOS at 0x%08x has signature 0x%04x\n",
>> +    DebugF("xf86ReadBIOS: BIOS at 0x%08lx has signature 0x%04x\n",
>>             Base, ptr[0] | (ptr[1] << 8));
>>      (void) memcpy(Buf, (void *) (ptr + Offset), Len);
>>      (void) munmap((caddr_t) ptr, mlen);
>
> i guess the caddr_t should read (void *)

Yes, it should, but that's a bug for another patch...


>> -        DB(printk("mem_read: address %#lx out of range!\n", addr);
>> +        DB(printk("mem_read: address %#x out of range!\n", addr);
>
> i guess %p is ok here also ? (same for below)

addr is a u32, so %p can't be used.

> Maybe you can use DebugF()  here also ?

x86emu has a #define DB(x) x, so with a change to fix warnings, we don't
want to also change semantics at all.

> just my 2 cents,

As you can see, just reviewing random bits of code always turns up more
things that could be fixed; if you want to do things like eliminate
caddr_t, that'd be awesome.

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140512/2b2d4e65/attachment-0001.sig>


More information about the xorg-devel mailing list