Update of bug #17884 (project mypaint):

                  Status:               Need Info => Confirmed              
             Assigned to:                    None => achadwick              

    _______________________________________________________

Follow-up Comment #4:

Confirmed: bad names in prefs "layout.window_positions" cause a failed
import.

The behaviour's not really any different from the previous "windowpos.conf"
versions, which also did a raw __import__ in response to stuff in a config
file. Let's at least catch the error and return the no-such-widget code:
http://gitorious.org/mypaint/mypaint/commit/fc5f11600265ca5f31a62b6107256c75e9690045

Whitelisting sounds like a good idea for security reasons, however.

Not sure how Edgar's problem came about. Unless an iportnov branch was using
"layout.window_positions" as a prefs key and doing something really similar,
it's probably not just that. I suspect a mis-merge somewhere back in history
that used the old static list of windows to load, and the new prefs block that
somehow ran and saved its prefs successfully...

    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?17884>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Mypaint-bugs mailing list
[email protected]
https://mail.gna.org/listinfo/mypaint-bugs

Reply via email to