[PATCH] powertop: possible a bug entered

[Denys Vlasenko]

On Thursday 04 November 2010 10:50, Maksym Kryzhanovskyy wrote:
> It seems that we need '\0' immediately after "  total events" in the buf, 
> otherwise strcmp does not match. See attachment.

Yes, indeed... do you think this simple fix is enough? -

                        count = skip_whitespace(buf);
-                       if (strcmp(strchrnul(count, ' '), " total events") == 0)
-                               break;
                        p = strchr(count, ',');
                        if (!p)
                                continue;
                        *p++ = '\0';
+                       if (strcmp(skip_non_whitespace(count), " total events") == 0)
+                               break;



-- 
vda