[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#749209: marked as done (kinput2: missing define may result in invalid memory access)

Your message dated Fri, 05 Apr 2024 17:47:53 +0000
with message-id <[🔎] E1rsnfN-00Fu8s-G2@fasolo.debian.org>
and subject line Bug#1068259: Removed package(s) from unstable
has caused the Debian Bug report #749209,
regarding kinput2: missing define may result in invalid memory access
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
749209: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=749209
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: kinput2
Version: 3.1-12
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 -o kinput2 -g -O2 -fno-strict-aliasing       kinput2.o ../lib/libKi2.a ../lib/imlib/libim.a  -lcanna16  -lXaw -lXmu -lXt -lSM -lICE -lXpm  -lXext -lX11      

error: conflicting function declarations "IMCloseConnection"
old definition in module IMProto file imlib/imfuncs.h line 70
void (struct _im_connection_ *)
new definition in module imdispatch file imfuncs.h line 70
void (struct _im_connection_ *)

reason for conflict at .serial in types listed below (struct/struct):
names of component 2 differ (serial/has_length_bug)
struct _im_connection_ {
  signed int major_protocol_version;
  signed int minor_protocol_version;
  signed int serial;
  unsigned int $pad0;
  struct _WidgetRec * proto_widget;
  IMTransport transport;
  signed int byte_order;
  unsigned int $pad1;
  IMBuffer in_buf;
  IMBuffer out_buf;
  signed int (*)() dispatcher;
  struct _im_im_ * im_list;
  signed int schedule;
  unsigned int $pad2;
  struct _im_connection_ * queue_next;
  struct _im_connection_ * next;
}
struct _im_connection_ {
  signed int major_protocol_version;
  signed int minor_protocol_version;
  signed int has_length_bug;
  signed int serial;
  struct _WidgetRec * proto_widget;
  IMTransport transport;
  signed int byte_order;
  unsigned int $pad1;
  IMBuffer in_buf;
  IMBuffer out_buf;
  signed int (*)() dispatcher;
  struct _im_im_ * im_list;
  signed int schedule;
  unsigned int $pad1;
  struct _im_connection_ * queue_next;
  struct _im_connection_ * next;
}
Makefile:1111: recipe for target 'kinput2' failed
make[2]: *** [kinput2] Error 64
make[2]: Leaving directory '/srv/jenkins-slave/workspace/sid-goto-cc-kinput2/kinput2-3.1/cmd'
Makefile:1099: recipe for target 'all' failed
make[1]: *** [all] Error 2

Looking at the struct layout above, it will depend on the size of padding
whether fields map onto each other or not. On 64 bit systems (as in this case),
the padding may help; on 32 bit systems the fields will be misaligned. In
particular, accesses such as conn->next

http://sources.debian.net/src/kinput2/3.1-12/lib/IMProto.c?hl=377#L377

may or may not be valid. This is all caused by -DXIM_BC not being part of all
compiler command lines.

Best,
Michael

Attachment: pgpfNXGYCeQug.pgp
Description: PGP signature


--- End Message ---
--- Begin Message --- 
Version: 3.1-14+rm

Dear submitter,

as the package kinput2 has just been removed from the Debian archive
unstable we hereby close the associated bug reports.  We are sorry
that we couldn't deal with your issue properly.

For details on the removal, please see https://bugs.debian.org/1068259

The version of this package that was in Debian prior to this removal
can still be found using https://snapshot.debian.org/.

Please note that the changes have been done on the master archive and
will not propagate to any mirrors until the next dinstall run at the
earliest.

This message was generated automatically; if you believe that there is
a problem with it please contact the archive administrators by mailing
ftpmaster@ftp-master.debian.org.

Debian distribution maintenance software
pp.
Thorsten Alteholz (the ftpmaster behind the curtain)

--- End Message ---
Reply to: