[SATYR PATCH] Use lower case for operating system names

Martin Milata mmilata at redhat.com
Tue May 21 14:19:14 UTC 2013


uReport should probably use an existing schema for operating system
names (see [1]). In the meantime, let's make the information from
/etc/system-release lowercase in order for faf to at least recognize
fedora.

Closes #77.

[1] https://github.com/abrt/satyr/issues/77

Signed-off-by: Martin Milata <mmilata at redhat.com>
---
 lib/operating_system.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/lib/operating_system.c b/lib/operating_system.c
index 9ff962b..cfb33bc 100644
--- a/lib/operating_system.c
+++ b/lib/operating_system.c
@@ -112,6 +112,14 @@ sr_operating_system_parse_etc_system_release(const char *etc_system_release,
     if (0 == strlen(*name))
         return false;
 
+    /* make the name all lower case */
+    char *a = *name;
+    while (*a)
+    {
+        *a = tolower(*a);
+        a++;
+    }
+
     const char *version_begin = release + strlen(" release ");
     const char *version_end = version_begin;
     while (isdigit(*version_end) || *version_end == '.')
-- 
1.7.11.7



More information about the Crash-catcher mailing list