This one time, at band camp, Falk Hueffner said:
> Hi,
> 
> is there a problem with this patch? Or why hasn't it been applied?

Well, this part looks good:

> --- libclamav/others.c~       2005-04-20 01:33:17.000000000 +0200
> +++ libclamav/others.c        2005-05-07 15:45:11.000000000 +0200
> -#if WORDS_BIGENDIAN == 0
> -    ret = *(int32_t *) buff;
> +#if WORDS_BIGENDIAN == 0 && defined(__GNUC__)
> +    ret = ((struct { int32_t x; } __attribute__((packed)) *) buff)->x;

But I am not so sure about this one:

> --- libclamav/upx.c~  2005-04-27 23:53:57.000000000 +0200
> +++ libclamav/upx.c   2005-05-07 16:14:46.000000000 +0200
> -#if WORDS_BIGENDIAN == 1
> +#ifndef __i386__
> -#if WORDS_BIGENDIAN == 0
> +#ifdef __i386__

Other architectures besides i386 are little endian, and I'm not sure
just swapping them is the right approach here.  If the problem to be
fixed is on one arch or 32 vs. 64 bit only, probably that should be
tested for instead.


Take care,
-- 
 -----------------------------------------------------------------
|   ,''`.                                            Stephen Gran |
|  : :' :                                        [EMAIL PROTECTED] |
|  `. `'                        Debian user, admin, and developer |
|    `-                                     http://www.debian.org |
 -----------------------------------------------------------------

Attachment: signature.asc
Description: Digital signature

Reply via email to