Bug #23529 compiler warning in mysys/my_getwd.c and other places
Submitted: 21 Oct 2006 21:37 Modified: 19 May 2008 11:22
Reporter: Hartmut Holzgraefe Email Updates:
Status: Can't repeat Impact on me:
None 
Category:MySQL Server: Compiling Severity:S3 (Non-critical)
Version:5.1bk OS:Any
Assigned to: Alexander Nozdrin CPU Architecture:Any

[21 Oct 2006 21:37] Hartmut Holzgraefe
Description:
my_getwd.c:40: warning: format ‘%lx’ expects type ‘long unsigned int’, but argument 2 has type ‘my_string’

How to repeat:
compile mysys/my_getwd.c with debug enabled

Suggested fix:
-  DBUG_PRINT("my",("buf: 0x%lx  size: %d  MyFlags %d", buf,size,MyFlags));
+  DBUG_PRINT("my",("buf: %p  size: %d  MyFlags %d", buf,size,MyFlags));
[21 Oct 2006 21:38] Bugs System
A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/14118

ChangeSet@1.2332, 2006-10-21 23:38:48+02:00, hartmut@mysql.com +1 -0
  a pointer is not an integer ... (Bug #23529)
[22 Oct 2006 14:57] Hartmut Holzgraefe
Integer format placeholders like %x and %lx are used for pointer arguments in a lot of places. There doesn't seem to be any portability problem with using %p though on our supported platforms as we're using it in other similar places already.
[22 Oct 2006 15:01] Bugs System
A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/14121

ChangeSet@1.2334, 2006-10-22 17:00:54+02:00, hartmut@mysql.com +61 -0
  %x / %lx -> %p format string changes for pointer arguments (Bug #23529)
[22 Oct 2006 23:36] Bugs System
A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/14124

ChangeSet@1.2336, 2006-10-23 01:36:58+02:00, hartmut@mysql.com +2 -0
  more %p / pointer placeholder fixes (Bug #23529)