dlt-receive: change part_num to increment only if rename was successful. (#593)

If the device got disconnected and you attempted to reconnect unsuccessfully, a file rename would happen. Then in the next successful connection, it would have one extra increment in the file's name when the file size limit was reached. This commit fixes this error.

Signed-off-by: Mike Konstantakos <[email protected]>

Author: Laximas

src/console/dlt-receive.c

@@ -284,14 +284,17 @@ int dlt_receive_open_output_file(DltReceiveData *dltdata)
         char filename[PATH_MAX + 1];
         filename[PATH_MAX] = 0;
 
-        snprintf(filename, PATH_MAX, "%s.%i.dlt", dltdata->ovaluebase, dltdata->part_num++);
+        snprintf(filename, PATH_MAX, "%s.%i.dlt", dltdata->ovaluebase,
+                 dltdata->part_num);
 
         if (rename(dltdata->ovalue, filename) != 0)
             dlt_vlog(LOG_ERR, "ERROR: rename %s to %s failed with error %s\n",
                      dltdata->ovalue, filename, strerror(errno));
-        else if (dltdata->vflag)
+        else if (dltdata->vflag) {
             dlt_vlog(LOG_INFO, "Renaming existing file from %s to %s\n",
                      dltdata->ovalue, filename);
+            ++dltdata->part_num;
+        }
     } /* if (file_already_exists) */
 
     globfree(&outer);