xbmc/xbmc — pull request #3807 by nigeljonez (2013-12-11 08:24:49+00:00)
Title: Avoid using CSettings to create a blank guisettings.xml for a fresh prof...

...ile. This causes a segmentation fault as documented in issue #14747.

As the file is created 'fresh' on first login of the profile, there seems to be no consequence in not creating before hand.

Relevant analysis & discussion over on trac at http://trac.xbmc.org/ticket/14747

A note that will be cross-posted to the Trac discussion, I had intended to include a second commit to avoid scaling div-by-zero issues, unfortunately simply falling back on a scale of 1.0f had a side-effect of screen blanking - most likely due to other rendering functions getting incorrect information about the size of the screen.

It seems the best path, is just to not let this happen, and after feature freeze consider separating the running m_resolutions configuration from CSettings (as opposed to user resolution preferences).