Changeset 13556 in vbox for trunk/src/VBox/HostServices/GuestProperties

Timestamp:

Oct 24, 2008 2:17:17 PM (17 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

38424

Message:

Guest Properties (Main, HostServices, VBoxGuestLib): MAX_*_LEN is the size in bytes, not the string length

File:

• trunk/src/VBox/HostServices/GuestProperties/service.cpp (modified) (4 diffs)

trunk/src/VBox/HostServices/GuestProperties/service.cpp

@@ -373 +373 @@
     uint32_t cchName, cchBuf;
     size_t cchValue, cchFlags, cchBufActual;
-    char szFlags[MAX_FLAGS_LEN + 1];
+    char szFlags[MAX_FLAGS_LEN];
     uint32_t fFlags;
 
@@ -649 +649 @@
     if (RT_SUCCESS(rc))
         rc = VBoxHGCMParmPtrGet(&paParms[0], (void **) &paszPatterns, &cchPatterns);
-    if (RT_SUCCESS(rc) && cchPatterns > MAX_PATTERN_LEN + 1)
+    if (RT_SUCCESS(rc) && cchPatterns > MAX_PATTERN_LEN)
         rc = VERR_TOO_MUCH_DATA;
     if (RT_SUCCESS(rc))
@@ -658 +658 @@
  */
     bool matchAll = false;
-    char pszPatterns[MAX_PATTERN_LEN + 1];
+    char pszPatterns[MAX_PATTERN_LEN];
     if (   (NULL == paszPatterns)
         || (cchPatterns < 2)  /* An empty pattern string means match all */
@@ -758 +758 @@
     uint64_t u64Timestamp = 0;
     uint32_t fFlags = NILFLAG;
-    char szFlags[MAX_FLAGS_LEN + 1];
+    char szFlags[MAX_FLAGS_LEN];
     char *pszName = NULL, *pszValue = NULL, *pszFlags = NULL;