Bug#993809: bullseye-pu: package segemehl/0.3.4-3+deb11u1 (Pre-approval)
On Tue, Sep 07, 2021 at 12:27:05AM +0530, Nilesh Patra wrote:
> diff -Nru segemehl-0.3.4/debian/patches/arm64.patch segemehl-0.3.4/debian/patches/arm64.patch
> --- segemehl-0.3.4/debian/patches/arm64.patch 1970-01-01 05:30:00.000000000 +0530
> +++ segemehl-0.3.4/debian/patches/arm64.patch 2021-09-06 23:43:50.000000000 +0530
> @@ -0,0 +1,75 @@
> +Description: Change the signed-ness for several chars to fix segfault
> +Author: Nilesh Patra <nilesh@debian.org>
> +Last-Update: 2021-08-24
> +--- a/libs/biofiles.c
> ++++ b/libs/biofiles.c
> +@@ -1916,7 +1916,7 @@
> + Uint max, Uint *minlen, Uint *maxlen, unsigned char *minq, unsigned char *maxq)
> + {
> +
> +- char ch;
> ++ signed char ch;
> + char idchar=0;
> + int ret=0;
> + off_t curseqoffset, lastindexoffset=0;
Shouldn't `ch` be an `int` instead, to match the return value of getc,
and type of EOF?
Anyway, I guess this is fine if it works...
Cheers,
Julien
Reply to: