summaryrefslogtreecommitdiff
path: root/qutebrowser/misc/sessions.py
diff options
context:
space:
mode:
authorFlorian Bruhin <me@the-compiler.org>2018-09-22 19:29:36 +0200
committerFlorian Bruhin <me@the-compiler.org>2018-09-22 19:29:36 +0200
commit22a4aaa73c5a3e1b76958b47a27c81f042ff05be (patch)
tree4f845e5f6057adc5880a1da4637384be0b0ffa17 /qutebrowser/misc/sessions.py
parent737db48c1476d2086395bff51894db605bd3481e (diff)
downloadqutebrowser-22a4aaa73c5a3e1b76958b47a27c81f042ff05be.tar.gz
qutebrowser-22a4aaa73c5a3e1b76958b47a27c81f042ff05be.zip
Handle empty session files gracefully
There's still much more to do for #1926, but with this we at least handle one common case gracefully. No tests because test_sessions.py is still in a kinda bad place...
Diffstat (limited to 'qutebrowser/misc/sessions.py')
-rw-r--r--qutebrowser/misc/sessions.py3
1 files changed, 3 insertions, 0 deletions
diff --git a/qutebrowser/misc/sessions.py b/qutebrowser/misc/sessions.py
index 358ab0c53..78adeb983 100644
--- a/qutebrowser/misc/sessions.py
+++ b/qutebrowser/misc/sessions.py
@@ -414,6 +414,9 @@ class SessionManager(QObject):
raise SessionError(e)
log.sessions.debug("Loading session {} from {}...".format(name, path))
+ if data is None:
+ raise SessionError("Got empty session file")
+
for win in data['windows']:
window = mainwindow.MainWindow(geometry=win['geometry'],
private=win.get('private', None))