[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 118793] Problematic characters break feed parsing
From:       Pau Capdevila <pau.capdevila () gmail ! com>
Date:       2006-01-31 22:07:52
Message-ID: 20060131220752.4689.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=118793         




------- Additional Comments From pau.capdevila gmail com  2006-01-31 23:07 \
------- I really admire you.

I hope one day I'll be half a programmer as you.

Thank you very much indeed!

Pau

On 31 Jan 2006 17:05:54 -0000, Frank Osterfeld <frank.osterfeld kdemail \
net> wrote:
[bugs.kde.org quoted mail]
I really admire you.<br><br>I hope one day I'll be half a programmer as \
you.<br><br>Thank you very much indeed!<br><br>Pau<br><br><div><span \
class="gmail_quote">On 31 Jan 2006 17:05:54 -0000, <b \
class="gmail_sendername">Frank Osterfeld </b> &lt;<a \
href="mailto:frank.osterfeld kdemail net">frank osterfeld kdemail \
net</a>&gt; wrote:</span><blockquote class="gmail_quote" \
style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt \
                0.8ex; padding-left: 1ex;">
------- You are receiving this mail because: -------<br>You reported the \
bug, or are watching the reporter.<br><br><a \
href="http://bugs.kde.org/show_bug.cgi?id=118793">http://bugs.kde.org/show_bug.cgi?id=118793</a><br>frank.osterfeld
  kdemail net changed:<br><br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \
What&nbsp;&nbsp;&nbsp;&nbsp;|Removed&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \
|Added<br>------------------------------------------------------------------ \
----------<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \
Status|UNCONFIRMED&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \
|RESOLVED <br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \
Resolution|&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp \
;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;|FIXED<br><br><br><br>------- \
Additional Comments From frank.osterfeld kdemail net&nbsp;&nbsp;2006-01-31 \
18:05 -------<br>SVN commit 504290 by osterfeld:<br><br>RSS parser: ignore \
unknown or invalid version attribute value in the &lt;rss&gt; tag and \
<br>just assume RSS 2.0. The older formats are compatible to 2.0, so this \
should work.<br>(at least better than refusing to parse the feeds)<br>BUG: \
118793<br><br><br><br> M&nbsp;&nbsp;+9 \
-0&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;ChangeLog<br> M&nbsp;&nbsp;+2 \
-1&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;src/librss/document.cpp \
<br><br><br>--- branches/KDE/3.5/kdepim/akregator/ChangeLog \
#504289:504290<br> @ -2,6 +2,15&nbsp;&nbsp; <br> ===================<br> \
(c) 2004-2006 the Akregator authors.<br><br>+Changes after \
1.2.1:<br>+----------------------------- <br>+<br>+Bug fixes:<br>+<br>+ \
2006/01/31 RSS parser: ignore unknown or invalid version attribute value in \
the &lt;rss&gt; tag and<br>+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;just \
assume RSS 2.0. The older formats are compatible to 2.0, so this should \
work. <br>+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;(at \
least better than refusing to parse the feeds) (#118793) -fo<br>+<br> \
Changes after 1.2:<br> -----------------------------<br><br>--- \
branches/KDE/3.5/kdepim/akregator/src/librss/document.cpp #504289:504290 \
<br> @ -110,7 +110,8&nbsp;&nbsp; \
<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \
d-&gt;version = v0_93;<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \
else if (attr == QString::fromLatin1(&quot;0.94&quot;))<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \
d-&gt;version = v0_94;<br>-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;else \
if (attr.startsWith(&quot;2.0&quot;) || attr == \
QString::fromLatin1(&quot;2&quot;)) //  <a \
href="http://www.breuls.org/rss">http://www.breuls.org/rss</a> puts 2.00 in \
version (BR #0000016)<br>+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;else \
// otherwise, we just assume a RSS2 compatible feed. As rss2 is \
generally<br>+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \
// backward-compatible, this should work \
<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \
d-&gt;version = v2_0;<br>&nbsp;&nbsp;&nbsp;&nbsp; \
}<br></blockquote></div><br>


[prev in list] [next in list] [prev in thread] [next in thread]