[PATCH 1/2] Update preference saving for numeric values
Linus Torvalds
torvalds at linux-foundation.org
Fri Jan 11 11:58:34 PST 2013
From: Linus Torvalds <torvalds at linux-foundation.org>
Date: Fri, 11 Jan 2013 11:37:32 -0800
Subject: [PATCH 2/2] Fix SIGSEGV if no pre-existing nickname information
Loading an xml file with dive computer information without any
preexisting nickname data would try to "remember" that NULL nickname.
Causing a SIGSEGV if there was no nickname information.
The bug was introduced by commit ec38d3708df2 ("Move device_info handling
into a new 'device.c' file") when I inadvertently removed a bit too much
code. It actually wants to remember a missing DC nickname as an empty one.
Signed-off-by: Linus Torvalds <torvalds at linux-foundation.org>
---
gtk-gui.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/gtk-gui.c b/gtk-gui.c
index 60dde03990a5..d096e7e90ccf 100644
--- a/gtk-gui.c
+++ b/gtk-gui.c
@@ -1737,6 +1737,9 @@ void remember_dc(const char *model, uint32_t deviceid, const char *nickname, gbo
if (!nn_entry)
return;
+ if (!nickname)
+ nickname = "";
+
/* No existing nickname? */
if (!nn_entry->nickname) {
char buffer[160];
More information about the subsurface
mailing list