Mailinglist Archive: opensuse-factory (443 mails)

< Previous Next >
Re: [opensuse-factory] New package 'opensuse-welcome' to Factory

On piÄ…, 12 lip, 2019 at 9:10 PM, Fabian Vogt <fvogt@xxxxxxx> wrote:

Am Mittwoch, 10. Juli 2019, 21:00:17 CEST schrieb Fabian Vogt:
> > I had quick look at the code and notices that it makes use of HTML and friends.
> > It seems to load RSS data over HTTP (no TLS) and shows the latest entry in the
> > page. If there's some vulnerability found in the RSS entry display, arbitrary
> > code can be executed using launcher.launch("foo").
> Changed to use HTTPS now.

IMO not enough, would it be possible to add some extra steps for sanitation?

So, I just had a more in-depth look and it's absolutely crazy how the RSS
display is implemented.

Instead of fetching the RSS XML document from directly,
it does a query to
which returns a javascript document, which is then directly eval'd in the
global context.

So by opening this application the user effectively gives
full access to his local user account, they just need to return
"launcher.launch('kdialog --msgbox Pwned')".

Alternatively, a post with the title containing
on would also lead to a similar result as there is
no HTML escaping whatsoever.

I reported those issues to jquery-rss upstream:

So please fix this before including the package in the distribution,
preferably by disabling all access to outside sources by default.

I feel a better solution would be to fetch news via C++ library (I will have to look what's around) and put that stuff in the slot. Doing all that with js is pointless. Well, js is pointless overall in that app, it's already barely used at all, and since we have all the access to the content via C++, might as well use that.

LCP [Stasiek]

To unsubscribe, e-mail: opensuse-factory+unsubscribe@xxxxxxxxxxxx
To contact the owner, e-mail: opensuse-factory+owner@xxxxxxxxxxxx

< Previous Next >
Follow Ups