Comment 9 for bug 72052

Revision history for this message
Reinhard Tartler (siretart) wrote : Re: [Bug 72052] MOTU SRU proposal

Lionel Le Folgoc <email address hidden> writes:

> --- pouetchess-0.2.0.orig/debian/patches/04-segfaults_fix.patch
> +++ pouetchess-0.2.0/debian/patches/04-segfaults_fix.patch
> @@ -0,0 +1,11 @@
> +--- src/faile/utils.c.orig 2006-05-28 00:32:00.000000000 +0200
> ++++ src/faile/utils.c 2006-11-15 23:31:39.000000000 +0100
> +@@ -125,7 +125,7 @@
> + rank (target) < 1 || rank (target) > 8 ||
> + file (target) < 1 || file (target) > 8)
> + {
> +- sprintf (str, "*** FAILE INTERNAL ERROR ***");
> ++ sprintf (str, "xxxx");
> + return;
> + }
> +

Could you please explain why replacing one string with a shorter one
fixes your bug?

Looking further, it seems that this bug is reported upstream as
http://sourceforge.net/tracker/index.php?func=detail&aid=1596684&group_id=157043&atid=802212. The
patch attached there is
http://sourceforge.net/tracker/download.php?group_id=157043&atid=802212&file_id=203026&aid=1596684

It looks to me that your patch to utils.c is only one part of what is
needed to fix the bug.

I'll update #63309 shortly.

I agree to uploading pouetchess to edgy-propsed with the complete
patch. Please mention this bug as well in the changelog, so that the
archive admins can see who actually approved the appload.

--
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4