----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114226/#review44907 -----------------------------------------------------------
This review has been submitted with commit 64a7238c3c082b174e47c3896d6eb0f5c52ab461 by Albert Astals Cid to branch KDE/4.11. - Commit Hook On Nov. 30, 2013, 6:40 p.m., Albert Astals Cid wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/114226/ > ----------------------------------------------------------- > > (Updated Nov. 30, 2013, 6:40 p.m.) > > > Review request for kdelibs and David Faure. > > > Bugs: 328182 > http://bugs.kde.org/show_bug.cgi?id=328182 > > > Repository: kdelibs > > > Description > ------- > > At the moment KLimitedIODevice::bytesAvailable is always returning the size > of the file, so QIODevice::atEnd never returns true because it thinks there > are more bytes to read. This makes that if you feed an "invalid" svg like > "<?xml version="1.0"?>" to the QImageReader it infinite loops believing there > will be more stuff to read but then read() always returns 0 but then > bytesAvailable says there are more and it stays there forever. > > > Diffs > ----- > > kdecore/io/klimitediodevice.cpp c93463b > > Diff: http://git.reviewboard.kde.org/r/114226/diff/ > > > Testing > ------- > > The svg renderer does not end loop anymore > > > Thanks, > > Albert Astals Cid > >