[Qemu-devel] [PATCH] configure: include stddef.h for NULL

[Qemu-devel] [PATCH] configure: include stddef.h for NULL

[Scott Wood]

This fixes an observed failure to detect madvise() on Linux.

To avoid similar issues, all other tests that use NULL but don't already
have stddef.h are also fixed.

Signed-off-by: Scott Wood <scottwood@freescale.com>
---
 configure |    9 +++++++++
 1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/configure b/configure
index e0d34fd..9c4f1db 100755
--- a/configure
+++ b/configure
@@ -1276,6 +1276,7 @@ if test "$vnc_sasl" != "no" ; then
   cat > $TMPC <<EOF
 #include <sasl/sasl.h>
 #include <stdio.h>
+#include <stddef.h>
 int main(void) { sasl_server_init(NULL, "qemu"); return 0; }
 EOF
   # Assuming Cyrus-SASL installed in /usr prefix
@@ -1319,6 +1320,7 @@ if test "$vnc_png" != "no" ; then
 cat > $TMPC <<EOF
 //#include <stdio.h>
 #include <png.h>
+#include <stddef.h>
 int main(void) {
     png_structp png_ptr;
     png_ptr = png_create_write_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL);
@@ -1503,6 +1505,7 @@ if test "$brlapi" != "no" ; then
   brlapi_libs="-lbrlapi"
   cat > $TMPC << EOF
 #include <brlapi.h>
+#include <stddef.h>
 int main( void ) { return brlapi__openConnection (NULL, NULL, NULL); }
 EOF
   if compile_prog "" "$brlapi_libs" ; then
@@ -1737,6 +1740,7 @@ if test "$linux_aio" != "no" ; then
   cat > $TMPC <<EOF
 #include <libaio.h>
 #include <sys/eventfd.h>
+#include <stddef.h>
 int main(void) { io_setup(0, NULL); io_set_eventfd(NULL, 0); eventfd(0, 0); return 0; }
 EOF
   if compile_prog "" "-laio" ; then
@@ -1759,6 +1763,7 @@ if test "$attr" != "no" ; then
 #include <stdio.h>
 #include <sys/types.h>
 #include <attr/xattr.h>
+#include <stddef.h>
 int main(void) { getxattr(NULL, NULL, NULL, 0); setxattr(NULL, NULL, NULL, 0, 0); return 0; }
 EOF
   if compile_prog "" "-lattr" ; then
@@ -1935,6 +1940,7 @@ cat > $TMPC << EOF
 #include <unistd.h>
 #include <fcntl.h>
 #include <limits.h>
+#include <stddef.h>
 
 int main(void)
 {
@@ -2032,6 +2038,7 @@ fi
 cat > $TMPC <<EOF
 #include <signal.h>
 #include <time.h>
+#include <stddef.h>
 int main(void) { clockid_t id; return clock_gettime(id, NULL); }
 EOF
 
@@ -2094,6 +2101,7 @@ madvise=no
 cat > $TMPC << EOF
 #include <sys/types.h>
 #include <sys/mman.h>
+#include <stddef.h>
 int main(void) { return madvise(NULL, 0, MADV_DONTNEED); }
 EOF
 if compile_prog "" "" ; then
@@ -2106,6 +2114,7 @@ fi
 posix_madvise=no
 cat > $TMPC << EOF
 #include <sys/mman.h>
+#include <stddef.h>
 int main(void) { return posix_madvise(NULL, 0, POSIX_MADV_DONTNEED); }
 EOF
 if compile_prog "" "" ; then
-- 
1.7.0.4

Re: [Qemu-devel] [PATCH] configure: include stddef.h for NULL

[Markus Armbruster]

Scott Wood <scottwood@freescale.com> writes:

> This fixes an observed failure to detect madvise() on Linux.
>
> To avoid similar issues, all other tests that use NULL but don't already
> have stddef.h are also fixed.

Appreciated, but several instances don't need fixing, because existing
includes already provide NULL.

> Signed-off-by: Scott Wood <scottwood@freescale.com>
> ---
>  configure |    9 +++++++++
>  1 files changed, 9 insertions(+), 0 deletions(-)
>
> diff --git a/configure b/configure
> index e0d34fd..9c4f1db 100755
> --- a/configure
> +++ b/configure
> @@ -1276,6 +1276,7 @@ if test "$vnc_sasl" != "no" ; then
>    cat > $TMPC <<EOF
>  #include <sasl/sasl.h>
>  #include <stdio.h>
> +#include <stddef.h>

stdio.h provides NULL.

>  int main(void) { sasl_server_init(NULL, "qemu"); return 0; }
>  EOF
>    # Assuming Cyrus-SASL installed in /usr prefix
> @@ -1319,6 +1320,7 @@ if test "$vnc_png" != "no" ; then
>  cat > $TMPC <<EOF
>  //#include <stdio.h>
>  #include <png.h>
> +#include <stddef.h>
>  int main(void) {
>      png_structp png_ptr;
>      png_ptr = png_create_write_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL);
> @@ -1503,6 +1505,7 @@ if test "$brlapi" != "no" ; then
>    brlapi_libs="-lbrlapi"
>    cat > $TMPC << EOF
>  #include <brlapi.h>
> +#include <stddef.h>
>  int main( void ) { return brlapi__openConnection (NULL, NULL, NULL); }
>  EOF
>    if compile_prog "" "$brlapi_libs" ; then
> @@ -1737,6 +1740,7 @@ if test "$linux_aio" != "no" ; then
>    cat > $TMPC <<EOF
>  #include <libaio.h>
>  #include <sys/eventfd.h>
> +#include <stddef.h>
>  int main(void) { io_setup(0, NULL); io_set_eventfd(NULL, 0); eventfd(0, 0); return 0; }
>  EOF
>    if compile_prog "" "-laio" ; then
> @@ -1759,6 +1763,7 @@ if test "$attr" != "no" ; then
>  #include <stdio.h>
>  #include <sys/types.h>
>  #include <attr/xattr.h>
> +#include <stddef.h>

Same here.

>  int main(void) { getxattr(NULL, NULL, NULL, 0); setxattr(NULL, NULL, NULL, 0, 0); return 0; }
>  EOF
>    if compile_prog "" "-lattr" ; then
> @@ -1935,6 +1940,7 @@ cat > $TMPC << EOF
>  #include <unistd.h>
>  #include <fcntl.h>
>  #include <limits.h>
> +#include <stddef.h>

unistd.h provides NULL.

>  
>  int main(void)
>  {
> @@ -2032,6 +2038,7 @@ fi
>  cat > $TMPC <<EOF
>  #include <signal.h>
>  #include <time.h>
> +#include <stddef.h>

time.h provides NULL.

>  int main(void) { clockid_t id; return clock_gettime(id, NULL); }
>  EOF
>  
> @@ -2094,6 +2101,7 @@ madvise=no
>  cat > $TMPC << EOF
>  #include <sys/types.h>
>  #include <sys/mman.h>
> +#include <stddef.h>
>  int main(void) { return madvise(NULL, 0, MADV_DONTNEED); }
>  EOF
>  if compile_prog "" "" ; then
> @@ -2106,6 +2114,7 @@ fi
>  posix_madvise=no
>  cat > $TMPC << EOF
>  #include <sys/mman.h>
> +#include <stddef.h>
>  int main(void) { return posix_madvise(NULL, 0, POSIX_MADV_DONTNEED); }
>  EOF
>  if compile_prog "" "" ; then

Re: [Qemu-devel] [PATCH] configure: include stddef.h for NULL

[Scott Wood]

On Tue, 5 Oct 2010 10:03:35 +0200
Markus Armbruster <armbru@redhat.com> wrote:

> Scott Wood <scottwood@freescale.com> writes:
> 
> > This fixes an observed failure to detect madvise() on Linux.
> >
> > To avoid similar issues, all other tests that use NULL but don't already
> > have stddef.h are also fixed.
> 
> Appreciated, but several instances don't need fixing, because existing
> includes already provide NULL.
> 
> > Signed-off-by: Scott Wood <scottwood@freescale.com>
> > ---
> >  configure |    9 +++++++++
> >  1 files changed, 9 insertions(+), 0 deletions(-)
> >
> > diff --git a/configure b/configure
> > index e0d34fd..9c4f1db 100755
> > --- a/configure
> > +++ b/configure
> > @@ -1276,6 +1276,7 @@ if test "$vnc_sasl" != "no" ; then
> >    cat > $TMPC <<EOF
> >  #include <sasl/sasl.h>
> >  #include <stdio.h>
> > +#include <stddef.h>
> 
> stdio.h provides NULL.

Hmm, I had looked up a reference to see if stdio.h is defined to
supply this, and didn't see it.  Today I see it. :-P

I'll respin the patch.

-Scott