Bug#1081126: gfpoken: FTBFS: ‘return’ with no value in function returning int

Santiago Vila sanvila at debian.org
Sun Sep 8 10:14:32 BST 2024


Package: src:gfpoken
Version: 1-4
Severity: serious
Tags: ftbfs

Dear maintainer:

During a rebuild of all packages in unstable, your package failed to build:

--------------------------------------------------------------------------------
[...]
main.c: In function ‘leveldraw_drag_end’:
main.c:681:17: error: ‘return’ with no value, in function returning non-void [-Wreturn-mismatch]
    681 |                 return;
        |                 ^~~~~~
main.c:678:10: note: declared here
    678 | gboolean leveldraw_drag_end ()
        |          ^~~~~~~~~~~~~~~~~~
main.c:684:17: error: ‘return’ with no value, in function returning non-void [-Wreturn-mismatch]
    684 |                 return;
        |                 ^~~~~~
main.c:678:10: note: declared here
    678 | gboolean leveldraw_drag_end ()
        |          ^~~~~~~~~~~~~~~~~~
make[2]: *** [Makefile:636: gfpoken-main.o] Error 1
make[2]: *** Waiting for unfinished jobs....
In file included from /usr/include/gtk-2.0/gtk/gtkobject.h:37,
                   from /usr/include/gtk-2.0/gtk/gtkwidget.h:36,
                   from /usr/include/gtk-2.0/gtk/gtkcontainer.h:35,
                   from /usr/include/gtk-2.0/gtk/gtkbin.h:35,
                   from /usr/include/gtk-2.0/gtk/gtkwindow.h:36,
                   from /usr/include/gtk-2.0/gtk/gtkdialog.h:35,
                   from /usr/include/gtk-2.0/gtk/gtkaboutdialog.h:32,
                   from /usr/include/gtk-2.0/gtk/gtk.h:33,
                   from gfp.h:21,
                   from multi.c:36:
/usr/include/gtk-2.0/gtk/gtktypeutils.h:236:1: warning: ‘GTypeDebugFlags’ is deprecated [-Wdeprecated-declarations]
    236 | void            gtk_type_init   (GTypeDebugFlags    debug_flags);
        | ^~~~
In file included from /usr/include/glib-2.0/gobject/gobject.h:26,
                   from /usr/include/glib-2.0/gobject/gbinding.h:31,
                   from /usr/include/glib-2.0/glib-object.h:24,
                   from /usr/include/glib-2.0/gio/gioenums.h:30,
                   from /usr/include/glib-2.0/gio/giotypes.h:30,
                   from /usr/include/glib-2.0/gio/gio.h:28,
                   from /usr/include/gtk-2.0/gdk/gdkapplaunchcontext.h:30,
                   from /usr/include/gtk-2.0/gdk/gdk.h:32,
                   from /usr/include/gtk-2.0/gtk/gtk.h:32:
/usr/include/glib-2.0/gobject/gtype.h:725:1: note: declared here
    725 | {
        | ^
In file included from /usr/include/gtk-2.0/gtk/gtktoolitem.h:31,
                   from /usr/include/gtk-2.0/gtk/gtktoolbutton.h:30,
                   from /usr/include/gtk-2.0/gtk/gtkmenutoolbutton.h:30,
                   from /usr/include/gtk-2.0/gtk/gtk.h:126:
/usr/include/gtk-2.0/gtk/gtktooltips.h:73:3: warning: ‘GTimeVal’ is deprecated: Use 'GDateTime' instead [-Wdeprecated-declarations]
     73 |   GTimeVal last_popdown;
        |   ^~~~~~~~
In file included from /usr/include/glib-2.0/glib/galloca.h:34,
                   from /usr/include/glib-2.0/glib.h:32,
                   from /usr/include/glib-2.0/gobject/gbinding.h:30:
/usr/include/glib-2.0/glib/gtypes.h:580:8: note: declared here
    580 | struct _GTimeVal
        |        ^~~~~~~~~
multi.c: In function ‘netsend’:
multi.c:314:3: warning: ignoring return value of ‘write’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
    314 |   write(remotesock, transdata, strlen(transdata));
        |   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
multi.c: In function ‘clientmonitor’:
multi.c:486:7: warning: ignoring return value of ‘write’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
    486 |       write(remotesock, newin, strlen(newin));
        |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
multi.c: In function ‘servermonitor’:
multi.c:616:7: warning: ignoring return value of ‘write’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
    616 |       write(remotesock, newin, strlen(newin));
        |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
multi.c:679:7: warning: ignoring return value of ‘write’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
    679 |       write(remotesock, newin, strlen(newin));
        |       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
multi.c: In function ‘accept_loop_timeout’:
multi.c:711:3: warning: ignoring return value of ‘write’ declared with attribute ‘warn_unused_result’ [-Wunused-result]
    711 |   write(remotesock, "GFingerPoken\n", 13);
        |   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
multi.c: In function ‘clientmonitor’:
multi.c:490:69: warning: ‘%s’ directive writing up to 77 bytes into a region of size 32 [-Wformat-overflow=]
    490 |     sprintf(newin, "Client: Server running an incompatible version, %s", netin+2);
        |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~
In file included from /usr/include/stdio.h:970,
                   from multi.c:29:
In function ‘sprintf’,
      inlined from ‘clientmonitor’ at multi.c:490:5:
/usr/include/x86_64-linux-gnu/bits/stdio2.h:30:10: note: ‘__builtin___sprintf_chk’ output between 49 and 126 bytes into a destination of size 80
     30 |   return __builtin___sprintf_chk (__s, __USE_FORTIFY_LEVEL - 1,
        |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     31 |                                   __glibc_objsize (__s), __fmt,
        |                                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     32 |                                   __va_arg_pack ());
        |                                   ~~~~~~~~~~~~~~~~~
multi.c: In function ‘servermonitor’:
multi.c:613:59: warning: ‘%s’ directive writing up to 77 bytes into a region of size 44 [-Wformat-overflow=]
    613 |       sprintf(newin, "Server: Incompatible client version %s",netin+2);
        |       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~
In function ‘sprintf’,
      inlined from ‘servermonitor’ at multi.c:613:7:
/usr/include/x86_64-linux-gnu/bits/stdio2.h:30:10: note: ‘__builtin___sprintf_chk’ output between 37 and 114 bytes into a destination of size 80
     30 |   return __builtin___sprintf_chk (__s, __USE_FORTIFY_LEVEL - 1,
        |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     31 |                                   __glibc_objsize (__s), __fmt,
        |                                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     32 |                                   __va_arg_pack ());
        |                                   ~~~~~~~~~~~~~~~~~
make[2]: Leaving directory '/<<PKGBUILDDIR>>'
dh_auto_build: error: make -j2 returned exit code 2
make[1]: *** [debian/rules:12: override_dh_auto_build] Error 25
make[1]: Leaving directory '/<<PKGBUILDDIR>>'
make: *** [debian/rules:6: build] Error 2
dpkg-buildpackage: error: debian/rules build subprocess returned exit status 2
--------------------------------------------------------------------------------

The above is just how the build ends and not necessarily the most relevant part.
If required, the full build log is available here:

https://people.debian.org/~sanvila/build-logs/202409/

About the archive rebuild: The build was made on virtual machines
of type m6a.large and r6a.large from AWS, using sbuild and a
reduced chroot with only build-essential packages.

If you could not reproduce the bug please contact me privately, as I
am willing to provide ssh access to a virtual machine where the bug is
fully reproducible.

If this is really a bug in one of the build-depends, please use
reassign and affects, so that this is still visible in the BTS web
page for this package.

[ Note: This is my third try at sending this bug. Apologies if you receive
   it by duplicate, there must be some aggressive spam filter in the BTS ]

Thanks.



More information about the Pkg-games-devel mailing list