On Thu, 2006-07-27 at 18:38 -0400, Jimi Xenidis wrote: > On Jul 27, 2006, at 6:21 PM, Hollis Blanchard wrote: > > > > It is a caller error to pass a NULL pointer and claim it holds >0 > > bytes. > > If this is a recurring problem we should add > > BUG_ON((buffer == NULL) && (bytes != 0)); > > The specific offender is: > arch/powerpc/platforms/xen/hcall.c HYPERVISOR_xen_version 95 case > XENVER_pagesize: > case XENVER_pagesize: > argsize = sizeof(void *); > break; > > so really we would want: > case XENVER_pagesize: > argsize = (arg == NULL) ? 0 : sizeof(void *); > break; > > Then your logic would be correct and we can add your BUG_ON() > statement above.
Sounds like a plan. :) -- Hollis Blanchard IBM Linux Technology Center _______________________________________________ Xen-ppc-devel mailing list Xen-ppc-devel@lists.xensource.com http://lists.xensource.com/xen-ppc-devel
