[AppInfo] OVT Coverity fixes.

The return value of Str_Snprintf is not being checked for negative
values and is directly passed to DynBuf_Append which accepts only
positive values. This may cause issues if Str_Snprintf fails. Added
a trivial check.

diff --git a/open-vm-tools/services/plugins/appInfo/appInfo.c b/open-vm-tools/services/plugins/appInfo/appInfo.c
index 774ccae9eb6f79dfcb4231c7f4e62510c5a7c0df..8801b68a097eb0b147fcc32078f75e61325ade1c 100644 (file)
--- a/open-vm-tools/services/plugins/appInfo/appInfo.c
+++ b/open-vm-tools/services/plugins/appInfo/appInfo.c
@@ -228,11 +228,14 @@ AppInfoGatherTask(ToolsAppCtx *ctx,
 
    tstamp = VMTools_GetTimeAsString();
 
-
    len = Str_Snprintf(tmpBuf, sizeof tmpBuf, headerFmt,
                       APP_INFO_VERSION_1,
                       counter,
                       tstamp != NULL ? tstamp : "");
+   if (len <= 0) {
+      g_warning("Insufficient space for the header.\n");
+      goto abort;
+   }
 
    DynBuf_Append(&dynBuffer, tmpBuf, len);