Fedora

SDL rendering issue: graphic corruption while scrolling right

Bug #725044 reported by cuc on 2011-02-25

This bug affects 3 people

	Status	Importance	Assigned to
The Battle For Wesnoth	Unknown	Unknown	gna.org #17573
libsdl	Fix Released	Medium	auto-bugzilla.libsdl.org #1090
Fedora	New	Undecided	Unassigned
Gentoo Linux	Fix Released	Medium	gentoo-bugs #357687
libsdl1.2 (Ubuntu)	Fix Released	Low	Unassigned
Natty	Fix Released	Low	Unassigned
wesnoth-1.8 (Ubuntu)	Invalid	Undecided	Unassigned
Natty	Invalid	Undecided	Unassigned

Bug Description

Binary package hint: wesnoth-1.8

using natty 11.04 with latest updates as of today (25.2)

when playing and scrolling to the right the screen gets corrupted, scrolling up/down/left works as expected
there is also a screenshot + a small diff available in the wesnoth forums ->
http://forums.wesnoth.org/viewtopic.php?uid=109471&f=4&t=32423&start=0

it happens using nouveau and nvidia binary driver..

Tags:

Related branches

lp://staging/~ubuntu-treblig/ubuntu/natty/libsdl1.2/libsdl1.2.fix-725044

Merged into lp://staging/ubuntu/natty/libsdl1.2 at revision 31

Ubuntu branches: Pending requested 2011-03-13

Revision history for this message

cuc (cuc+) wrote on 2011-02-25:

https://gna.org/bugs/index.php?17573

Revision history for this message

In Gentoo Bugzilla #357687, Richard (shiningarcanine) wrote on 2011-03-06:

There is an upstream bug report regarding this that contains a patch:

http://bugzilla.libsdl.org/show_bug.cgi?id=1090

Bug #354175 involves a program that is affected by this. Applying the upstream patch to media-libs/libsdl-1.2.14-r5 resolves it. The following commands can be used to test the patch:

ebuild $(equery which media-libs/libsdl-1.2.14-r5) prepare
cd /var/tmp/portage/media-libs/libsdl-1.2.14-r5/work/SDL-1.2.14
wget -O - http://bugzilla.libsdl.org/attachment.cgi?id=574 | patch --dry-run -p1
ebuild $(equery which media-libs/libsdl-1.2.14-r5) merge

The other ebuild versions, including the media-libs/libsdl-1.2.13-r1 ebuild used by stable, are also affected. The following commands can be used to test the patch with media-libs/libsdl-1.2.13-r1:

ebuild $(equery which media-libs/libsdl-1.2.13-r1) prepare
cd /var/tmp/portage/media-libs/libsdl-1.2.13-r1/work/SDL-1.2.13
wget -O - http://bugzilla.libsdl.org/attachment.cgi?id=574 | patch --dry-run -p1
ebuild $(equery which media-libs/libsdl-1.2.13-r1) merge

The reason we are seeing this issue in programs that rely on libsdl now is because of a change to glibc's memcpy() function late last year. The old implementation allowed programs to call memcpy() on overlapping regions. The new implementation copies memory in reverse, which broke any code that relied on the old implementation. The assumption that memcpy() works between overlapping regions of memory is illegal under ANSI C. There is a fairly lengthy bug report at the Fedora Linux bug tracker about this where Linus Torvalds commented on the issue:

https://bugzilla.redhat.com/show_bug.cgi?id=638477

I recommend inserting media-libs/libsdl-1.2.13-r2 and media-libs/libsdl-1.2.14-r6 into the portage tree with this patch. I also strongly recommend removing the existing ebuilds from the portage tree.

Revision history for this message

In Gentoo Bugzilla #357687, Richard (shiningarcanine) wrote on 2011-03-06:

I made a small mistake in my previous comment. The commands to test the application of this patch to unstable should be:

ebuild $(equery which media-libs/libsdl-1.2.14-r5) prepare
cd /var/tmp/portage/media-libs/libsdl-1.2.14-r5/work/SDL-1.2.14
wget -O - http://bugzilla.libsdl.org/attachment.cgi?id=574 | patch -p1
ebuild $(equery which media-libs/libsdl-1.2.14-r5) merge

The commands to test the application of this patch to stable should be:

ebuild $(equery which media-libs/libsdl-1.2.13-r1) prepare
cd /var/tmp/portage/media-libs/libsdl-1.2.13-r1/work/SDL-1.2.13
wget -O - http://bugzilla.libsdl.org/attachment.cgi?id=574 | patch -p1
ebuild $(equery which media-libs/libsdl-1.2.13-r1) merge

Sorry about any confusion my mistake might have caused. I had already modified the ebuild in a local overlay, so when I was testing these commands, I had to first reverse the patch and then apply it. I accidentally copied one of the --dry-run commands I had done during the course of this and then carried that mistake over the other set I had posted.

Revision history for this message

In Gentoo Bugzilla #357687, Richard (shiningarcanine) wrote on 2011-03-06:

I just noticed that sys-libs/glibc-2.13-r1 is keyworded, so anyone affected by this is likely using the unstable tree. Please disregard the instructions I posted for users of the stable tree. As long as they do not unkeyword glibc, they will likely not need to worry about this.

Revision history for this message

In Gentoo Bugzilla #357687, Richard (shiningarcanine) wrote on 2011-03-06:

(In reply to comment #2)
> I just noticed that sys-libs/glibc-2.13-r1 is keyworded, so anyone affected by
> this is likely using the unstable tree. Please disregard the instructions I
> posted for users of the stable tree. As long as they do not unkeyword glibc,
> they will likely not need to worry about this.
>

Actually disregard comment #2. That was meant for bug #354175. I posted it in this bug by mistake.

Revision history for this message

In Gentoo Bugzilla #357687, Mr-bones- (mr-bones-) wrote on 2011-03-08:

#10

fixed in libsdl-1.2.14-r6

Revision history for this message

In Gentoo Bugzilla #357687, Richard (shiningarcanine) wrote on 2011-03-09:

#11

(In reply to comment #4)
> fixed in libsdl-1.2.14-r6

libsdl-1.2.13-r1 also misuses memcpy(), so you might want to patch it too. Otherwise, this issue will occur again when glibc-2.13 becomes stable.

Revision history for this message

In Gentoo Bugzilla #357687, Mr-bones- (mr-bones-) wrote on 2011-03-09:

#12

guess we'll mark the later version of libsdl stable before making glibc-2.13 stable.

Revision history for this message

Dave Gilbert (ubuntu-treblig) wrote on 2011-03-12:

srk9 on #ubuntu-devel says this is a memcpy v memmove issue in libsdl and that it just needs a trivial fix from:

http://bugzilla.libsdl.org/show_bug.cgi?id=1090

this is likely to impact multiple SDL apps with odd corruption on x86.

Dave

Revision history for this message

Richard (shiningarcanine) wrote on 2011-03-13:

This is caused by an optimization introduced in glibc that causes memcpy() to copy in reverse. This breaks code that used memcpy() for overlapping copies in violation of ANSI C. There is a bug at the Fedora bug tracker documenting this where Linus Torvalds posted his two cents:

https://bugzilla.redhat.com/show_bug.cgi?id=638477

Anyway, the fix is to apply the patch at the libsdl bug tracker to libsdl. While the fix is trivial, it would be problematic if a major Linux distribution shipped a release with this bug in it. I suggest that this be made a blocker to Ubuntu 11.04's release to ensure that this is addressed before Ubuntu 11.04 ships next month.

Also, would the package maintainer take the initiative to notify his Debian counterpart? I am the Gentoo Linux user that filed the Gentoo bug about this and I don't plan to notify many more people. The only reason I cared about this bug on Ubuntu to talk about it here is because it affects someone I know.

Revision history for this message

Dave Gilbert (ubuntu-treblig) wrote on 2011-03-13:

I can confirm the fix at:

http://hg.libsdl.org/SDL/rev/d898ee5431f5

fixes this issue from a simple test in wesnoth.

Dave

summary:	- graphic corruption while scrolling right + SDL rendering issue: graphic corruption while scrolling right
Changed in wesnoth-1.8 (Ubuntu):
status:	New → Invalid
tags:	added: patch

Revision history for this message

Richard (shiningarcanine) wrote on 2011-03-13:

The bug has not been filed with Fedora yet, although the original upstream bug with the Battle for Wesnoth project was reported by a Fedora user.

Changed in fedora:
importance:	Unknown → Undecided
status:	Unknown → New

Andreas Moog (ampelbein) on 2011-03-13

Changed in libsdl1.2 (Ubuntu):
importance:	Undecided → Low
status:	New → Triaged

Bug Watch Updater (bug-watch-updater) on 2011-03-13

Changed in gentoo:
importance:	Unknown → Medium
status:	Unknown → Fix Released

Revision history for this message

Richard (shiningarcanine) wrote on 2011-03-19:

#13

Battle for Wesnoth implemented a workaround for the libsdl issue in the 1.8.x branch in svn commit r48934 and the 1.9.x trunk in svn commit r48935. It should be available for the general public in the 1.8.6 maintenance release.

Revision history for this message

Launchpad Janitor (janitor) wrote on 2011-03-31:

#14

This bug was fixed in the package libsdl1.2 - 1.2.14-6.1ubuntu3

---------------
libsdl1.2 (1.2.14-6.1ubuntu3) natty; urgency=low

  * Replace memcpy by memmove in SDL_blit.c, as per SDL bug 1090
    see http://hg.libsdl.org/SDL/rev/d898ee5431f5
    (LP: #725044).
-- <email address hidden> (Dr. David Alan Gilbert) Sun, 13 Mar 2011 01:05:52 +0000

Changed in libsdl1.2 (Ubuntu Natty):
status:	Triaged → Fix Released

Bug Watch Updater (bug-watch-updater) on 2012-11-21

Changed in libsdl:
importance:	Unknown → Medium
status:	Unknown → Fix Released

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

gentoo-bugs #357687
[RESOLVED FIXED] Edit
redhat-bugs #638477
[CLOSED ERRATA] Edit
gna.org #17573 Edit
auto-bugzilla.libsdl.org #1090
[RESOLVED FIXED] Edit

Bug watches keep track of this bug in other bug trackers.