Package: kluppe
Version: 0.6.14-2
Usertags: goto-cc

During a rebuild of all packages in a clean sid chroot (and cowbuilder+pbuilder)
the build failed with the following error. Please note that we use our research
compiler tool-chain (using tools from the cbmc package), which permits extended
reporting on type inconsistencies at link time.

[...]
gcc -pthread -Wall -DHAVE_ALSA -DHAVE_USB -DHAVE_OSC -g -O2 -fstack-protector 
--param=ssp-buffer-size=4 -Wformat -Werror=format-security 
-DPIXMAPS_DIR=\"/usr/share/pixmaps\" *.o ../../common/*.o -o kluppe `pkg-config 
gtk+-2.0 gthread-2.0 libusb alsa jack sndfile liblo libxml-2.0 --libs 
gthread-2.0` -lm -Wl,-z,relro

file serialio.c line 13: error: conflicting types for variable "status"
old definition in module kluppe file kluppe.c line 50
struct _GtkWidget {
  struct _GtkObject object;
  unsigned short int private_flags;
  unsigned char state;
  unsigned char saved_state;
  unsigned int $pad0;
  char * name;
  struct _GtkStyle * style;
  struct _GtkRequisition requisition;
  struct _GdkRectangle allocation;
  struct _GdkDrawable * window;
  struct _GtkWidget * parent;
} *
new definition in module serialio file serialio.c line 13
signed int
Makefile:8: recipe for target 'kluppe' failed
make[3]: *** [kluppe] Error 64
make[3]: Leaving directory 
'/srv/jenkins-slave/workspace/sid-goto-cc-kluppe/kluppe-0.6.14/src/frontend/kluppe'
Makefile:13: recipe for target 'kluppe' failed
make[2]: *** [kluppe] Error 2

Sharing the storage as both a pointer and an int would be bound to fail in
dangerous ways; the only reason this supposedly works right now is that all code
using "status" in serialio.c *also* declares function-local variables "status"
that shadow the global one.

The appropriate fix should be removing the global status variable from
serialio.c.

Best,
Michael

Attachment: pgpRYKn2YsB7H.pgp
Description: PGP signature

Reply via email to