Changeset 3833 in MondoRescue for branches/3.3/mondo/src/common

Timestamp:

Mar 5, 2024, 1:24:48 AM (4 months ago)

Author:

Bruno Cornec

Message:

manages tmpdir dynamically

Location:

branches/3.3/mondo/src/common

Files:

• libmondo-tools.c (modified) (3 diffs)
• mondostructures.h (modified) (1 diff)
• newt-specific.c (modified) (2 diffs)

branches/3.3/mondo/src/common/libmondo-tools.c

@@ -710 +710 @@
 
     char *tmp = NULL;
-    char *p = NULL;
-
-    if (bkpinfo->tmpdir[0] != '\0') {
+
+    if (bkpinfo->tmpdir != NULL) {
         /* purging a potential old tmpdir */
         if (chdir("/tmp")) {
@@ -744 +743 @@
         mr_asprintf(tmp, "/tmp/mondo.tmp.XXXXXX");
     }
-    p = mkdtemp(tmp);
-    if (p == NULL) {
+    bkpinfo->tmpdir = mkdtemp(tmp);
+    if (bkpinfo->tmpdir == NULL) {
         printf("Failed to create global tmp directory %s for Mondo.",tmp);
         mr_free(tmp);
         finish(-1);
     }
-    strcpy(bkpinfo->tmpdir,p);
-    mr_free(tmp);
-
     log_it("bkpinfo->tmpdir is being set to %s", bkpinfo->tmpdir);
-
     mr_system("mkdir -p %s/tmpfs", bkpinfo->tmpdir);
 }
@@ -833 +828 @@
     
     /*  We need tmpdir as early as possible for further function calls */
-    bkpinfo->tmpdir[0] = '\0';          // Really setup after
+    bkpinfo->tmpdir = NULL;         // Really setup after
     bkpinfo->scratchdir[0] = '\0';          // Really setup after
     setup_tmpdir(NULL);

branches/3.3/mondo/src/common/mondostructures.h

@@ -489 +489 @@
    * of space here.
    */
-    char tmpdir[MAX_STR_LEN / 4];
+    char *tmpdir;
 
   /**

branches/3.3/mondo/src/common/newt-specific.c

@@ -402 +402 @@
             mr_free(command);
         }
-        if (bkpinfo->tmpdir) {
+        if (bkpinfo->tmpdir != NULL) {
             log_msg(8,"tempdir is %s",bkpinfo->tmpdir);
             if (strstr(bkpinfo->tmpdir ,"mondo.tmp.") != NULL) {
@@ -439 +439 @@
         mr_free(bkpinfo->zip_exe);
         mr_free(bkpinfo->zip_suffix);
+        mr_free(bkpinfo->tmpdir);
         /* Then free the structure */
         paranoid_free(bkpinfo);