fixed bugs in LDAP ParseRange
authorVictor Kirhenshtein <victor@netxms.org>
Sat, 14 May 2016 14:41:37 +0000 (17:41 +0300)
committerVictor Kirhenshtein <victor@netxms.org>
Sat, 14 May 2016 14:41:37 +0000 (17:41 +0300)
src/server/core/ldap.cpp

index 1141a23..c9760c4 100644 (file)
@@ -1,6 +1,6 @@
 /*
 ** NetXMS - Network Management System
-** Copyright (C) 2003-2015 Raden Solutions
+** Copyright (C) 2003-2016 Raden Solutions
 **
 ** This program is free software; you can redistribute it and/or modify
 ** it under the terms of the GNU General Public License as published by
@@ -624,30 +624,33 @@ TCHAR *LDAPConnection::getAttrValue(LDAPMessage *entry, const char *attr, UINT32
    return result;
 }
 
+/**
+ * Parse range information from attribute
+ */
 static void ParseRange(const char *attr, int *start, int *end)
 {
    *end  = -1;
    *start = -1;
-   //read from ';' till '-' and till end
-   char *tmpAttr = strdup(attr);
-   char *tmpS = strchr(tmpAttr,'=');
-   char *tmpE = strchr(tmpAttr,'-');
-   if(tmpS == NULL || tmpE == NULL)
-      return;
-   *tmpE = 0;
-   *start = atoi(tmpS);
-   tmpS = tmpE + 1;
-   if(tmpS == NULL)
+
+   char *tmpS = strchr(attr, '=');
+   if (tmpS == NULL)
       return;
-   if(tmpS[0] == '*')
+
+   char *tmpAttr = strdup(tmpS + 1);
+   char *tmpE = strchr(tmpAttr, '-');
+   if (tmpE == NULL)
    {
-      *end  = -1;
+      free(tmpAttr);
+      return;
    }
-   else
+   *tmpE = 0;
+   tmpE++;
+   *start = atoi(tmpAttr);
+   if (tmpE[0] != '*')
    {
-      *end = atoi(tmpS);
+      *end = atoi(tmpE);
    }
-   safe_free(tmpAttr);
+   free(tmpAttr);
 }
 
 /**