Author: ekohl
Date: Mon Jul  4 21:04:00 2016
New Revision: 71813

URL: http://svn.reactos.org/svn/reactos?rev=71813&view=rev
Log:
[SC]
- Fix buggy actions parser.
- Use proper NUL character instead of zero.

Modified:
    trunk/reactos/base/applications/sc/misc.c

Modified: trunk/reactos/base/applications/sc/misc.c
URL: 
http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/sc/misc.c?rev=71813&r1=71812&r2=71813&view=diff
==============================================================================
--- trunk/reactos/base/applications/sc/misc.c   [iso-8859-1] (original)
+++ trunk/reactos/base/applications/sc/misc.c   [iso-8859-1] Mon Jul  4 
21:04:00 2016
@@ -189,16 +189,16 @@
 
     /* Replace all slashes by null characters */
     p = pStringBuffer;
-    while (*p != 0 /*_T('\0')*/)
+    while (*p != _T('\0'))
     {
         if (*p == _T('/'))
-            *p = 0; //_T('\0');
+            *p = _T('\0');
         p++;
     }
 
     /* Count the arguments in the buffer */
     p = pStringBuffer;
-    while (*p != 0 /*_T('\0')*/)
+    while (*p != _T('\0'))
     {
         nCount++;
 
@@ -224,11 +224,11 @@
         if (nCount % 2 == 0)
         {
             /* Action */
-            if (_tcsicmp(p, _T("reboot")))
+            if (!lstrcmpi(p, _T("reboot")))
                 pActions[nCount / 2].Type = SC_ACTION_REBOOT;
-            else if (_tcsicmp(p, _T("restart")))
+            else if (!lstrcmpi(p, _T("restart")))
                 pActions[nCount / 2].Type = SC_ACTION_RESTART;
-            else if (_tcsicmp(p, _T("run")))
+            else if (!lstrcmpi(p, _T("run")))
                 pActions[nCount / 2].Type = SC_ACTION_RUN_COMMAND;
             else
                 break;
@@ -262,7 +262,7 @@
     OUT LPCTSTR *ppServiceName,
     OUT LPSERVICE_FAILURE_ACTIONS pFailureActions)
 {
-    INT /*i,*/ ArgIndex = 1;
+    INT ArgIndex = 1;
     LPCTSTR lpActions = NULL;
     LPCTSTR lpReset = NULL;
 


Reply via email to