If $LANG isn't set, mythbackend segfaults after v0.25pre-3161-g0fd3987

[Bill Meek <keemllib@…>]

Running on Ubuntu 10.04.3 LTS, starting mythbackend from upstart, LANG is not set. 0fd3987 adds a check for it, but segfaults if it is not set.

/var/log/mythtv/mythbackend.*.PID.log size is 0.

The following patch eliminates the segfault and allows the error message to print.

diff --git a/mythtv/libs/libmythbase/mythcorecontext.cpp b/mythtv/libs/libmythbase/mythcorecontext.cpp
index 960ccbb..f3ddfbc 100644
--- a/mythtv/libs/libmythbase/mythcorecontext.cpp
+++ b/mythtv/libs/libmythbase/mythcorecontext.cpp
@@ -194,7 +194,7 @@ bool MythCoreContext::Init(void)
 #ifndef _WIN32
     char *lang = getenv("LANG");
-    if (!strcmp(lang, "C") || (strlen(lang) == 0))
+    if (lang == 0 || !strcmp(lang, "C") || (strlen(lang) == 0))
         LOG(VB_GENERAL, LOG_WARNING,
                 "This application expects to be running a UTF locale, and "
                 "many features may behave improperly with your current LANG "

[Bill Meek <keemllib@…>]

Version

[Bill Meek <keemllib@…>]

Backtrace

[Github]

Deal with a missing LANG env variable on startup

Although this really should be fixed in the startup script, we should not be crashing if someone doesn't set LANG.

Fixes #10005

Signed-off-by: Gavin Hurlbut <ghurlbut@…>

Branch: master Changeset: da6be99c9d9770424fe82056f3d8e0c9ad7af708

[beirdo]

Bill: you really need to fix your startup script to set LANG to a UTF-8 language if you want the MythTV setup to work correctly. Thanks for the crash-prevention, though.

[Bill Meek <keemllib@…>]

Replying to beirdo:

Gavin, thanks for the quick response! I tested my patch with and without the upstart script fix. Tomorrow I'll make an addition to: http://www.mythtv.org/wiki/Upstart_mythbackend_Configuration reflecting the new check.