[OpenJDK 2D-Dev] [8] request for review: 7150594: VM chash in JCK api/java_awt/Image/ConvolveOp/ tests for 64 bit jdk8 on linux.

Andrew Brygin andrew.brygin at oracle.com
Thu Aug 2 14:36:55 UTC 2012


Hello,

  could you please review a fix for 7150594?

  This problem is triggered by the fix for CR 7113017. In particular,
  this fix replaces the malloc.h with stddef.h in mlib_types.h. This
  change leads to compiling mlib_sys.c without forward declaration for
  memalign() routine, and cause following warnings:

mlib_sys.c:96: warning: implicit declaration of function 'memalign'
mlib_sys.c:96: warning: cast to pointer from integer of different size

  This cause the problem on systems where size of integer is less than
  size of pointers: the pointer value is clamped, and usage of the
  clamped pointer causes the observed crash.

  Proposed solution is to include malloc.h header directly to mlib_sys.c.

  Please take a look.

Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7150594
Webrev: http://cr.openjdk.java.net/~bae/7150594/8/webrev.00/

Thanks,
Andrew



More information about the 2d-dev mailing list