tags 323425 -patch
done

dann frazier wrote:
Package: cronolog
Version: 1.6.2-5
Severity: important
Tags: patch

Our automated buildd log filter[1] detected a problem[2] that will cause
your package to segfault on architectures where the size of a pointer is
greater than the size of an integer, such as ia64.

_XOPEN_SOURCE must be defined when using the strptime() function.  See
strptime(3) for details.  There might be a more autoconfy patch, but
this one works.

[1]http://people.debian.org/~dannf/check-implicit-pointer-functions
[2]Function `strptime' implicitly converted to pointer at cronoutils.c:673

I have applied a fix to the package, and stand duly chastized for having failed to follow up on it in a more useful fashion. Mea culpa. However, the patch provided is not the most appropriate way to fix the problem; since it involves adding a -D flag to CFLAGS, this is perfectly capable of happening in the debian/rules file, rather than requiring a source patch (I guess you could call that 'more autoconfy'; certainly, it seems cleaner).

However, since I have changed the behavior, it would be helpful if I could arrange to validate that the -6 package will, in fact, correctly solve this problem on a 64-bit architecture. Since I don't particularly trust that I can verify this sufficiently, myself, would it be possible to get a test build verified in some fashion?
--
Joel Aelwyn <[EMAIL PROTECTED]>                                       ,''`.
                                                                     : :' :
                                                                     `. `'
                                                                       `-

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to