Resolved SDOCM00112181
authorJustin Sobota <jsobota@ti.com>
Tue, 22 Jul 2014 20:34:54 +0000 (16:34 -0400)
committerJustin Sobota <jsobota@ti.com>
Tue, 22 Jul 2014 20:34:54 +0000 (16:34 -0400)
docs/ReleaseNotes_RM.doc
docs/ReleaseNotes_RM.pdf
include/rm_internal.h
src/rm_policy.c

index 8af02bcbf2aecfbbd6f48bab30f4a1cbe2adfcc7..9b88cdf1d86a9fb5ba1e02c86d2c6e6171f60b29 100644 (file)
Binary files a/docs/ReleaseNotes_RM.doc and b/docs/ReleaseNotes_RM.doc differ
index 1e6031c296e53de3049dd67c1e20e6ae9943dc70..f2d04276b326824dcf1c0b7ca1b19a55e8d8da9c 100644 (file)
Binary files a/docs/ReleaseNotes_RM.pdf and b/docs/ReleaseNotes_RM.pdf differ
index deba58db3d8e6729af9d8d61cda01d809b91d784..ce07e38b70d462f0839ca4c0ec1433c915747d1a 100644 (file)
@@ -114,7 +114,7 @@ typedef struct {
     char     nameServerName[RM_NAME_MAX_CHARS];
 } Rm_ResourceInfo;
 
-/* wrapper to strncpy to avoid covery overrun warning */
+/* wrapper to strncpy to avoid coverity overrun warning */
 static inline void rm_strncpy (char *dst, const char *src, size_t len)
 {
     strncpy (dst, src, len);
index 441cd912020832a94ab795d5bf5e4ac1a20b26c8..bef56965848d97c520c3e152198e4787e2d49114 100644 (file)
@@ -144,8 +144,6 @@ static Rm_PolicyPermission *policyParseSubPermission(char *permStrStart, char *p
     /* Create a local copy of the sub-permission string */
     permStrPtr = Rm_osalMalloc(permStrLen);
     rm_strncpy(permStrPtr, permStrStart, permStrLen);
-    /* Make sure the last character in the copied sub-permission string is null */
-    permStrPtr[permStrLen - 1] = '\0';
 
     permStrStart = permStrPtr;
     permStrEnd = permStrPtr + strlen(permStrPtr);
@@ -369,7 +367,7 @@ static Rm_PolicyPermission *policyParseSubPermission(char *permStrStart, char *p
         permStrPtr++;
     }
 
-    Rm_osalFree((void *)permStrStart, sizeof(permStrLen));
+    Rm_osalFree((void *)permStrStart, permStrLen);
     *result = RM_OK;
     return (startPerm);
 
@@ -379,7 +377,7 @@ parseError:
         Rm_osalFree((void *)startPerm, sizeof(Rm_PolicyPermission));
         startPerm = nextPerm;
     }    
-    Rm_osalFree((void *)permStrStart, sizeof(permStrLen));
+    Rm_osalFree((void *)permStrStart, permStrLen);
     return(NULL);     
 }