If str(n)dup fails, there is no more memory left. We should exit with "Out of memory". --- init.c | 3 +++ sysfs.c | 8 ++++++++ 2 files changed, 11 insertions(+) --- lm-sensors-dev/lib/init.c 2009-02-18 11:47:34.000000000 +0100 +++ my-sensors/lib/init.c 2009-02-19 22:04:32.000000000 +0100 @@ -53,6 +53,9 @@ static int sensors_parse(void) locale = setlocale(LC_ALL, NULL); if (locale) { locale = strdup(locale); + if (!locale) + sensors_fatal_error(__func__, "Out of memory"); + setlocale(LC_ALL, "C"); } --- lm-sensors-dev/lib/sysfs.c 2009-02-07 16:47:34.000000000 +0100 +++ my-sensors/lib/sysfs.c 2009-02-19 22:02:45.000000000 +0100 @@ -188,9 +188,14 @@ char *get_feature_name(sensors_feature_t case SENSORS_FEATURE_CURR: underscore = strchr(sfname, '_'); name = strndup(sfname, underscore - sfname); + if (!name) + sensors_fatal_error(__func__, "Out of memory"); + break; default: name = strdup(sfname); + if (!name) + sensors_fatal_error(__func__, "Out of memory"); } return name; @@ -419,6 +424,9 @@ static int sensors_read_dynamic_chip(sen /* fill in the subfeature members */ all_subfeatures[i].type = sftype; all_subfeatures[i].name = strdup(name); + if (!all_subfeatures[i].name) + sensors_fatal_error(__func__, "Out of memory"); + if (!(sftype & 0x80)) all_subfeatures[i].flags |= SENSORS_COMPUTE_MAPPING; all_subfeatures[i].flags |= sensors_get_attr_mode(dev_path, name);