Changeset 2303 in MondoRescue for branches/2.2.10/mondo/src/common/libmondo-stream.c

Timestamp:

Jul 24, 2009, 9:19:38 PM (11 years ago)

Author:

Bruno Cornec

Message:

find_tape_device_and_size => mr_find_tape_device which allocates 1 string to be freed by caller

File:

• branches/2.2.10/mondo/src/common/libmondo-stream.c (modified) (4 diffs)

branches/2.2.10/mondo/src/common/libmondo-stream.c

@@ -250 +250 @@
 
 /**
- * Determine the name and size of the tape device. Tries the SCSI tape for
+ * Determine the name of the tape device. Tries the SCSI tape for
  * this platform, then the IDE tape, then "/dev/st0", then "/dev/osst0".
- * @param dev Where to put the found tape device.
- * @param siz Where to put the tape size (a string like "4GB")
- * @return 0 if success, nonzero if failure (in which @p dev and @p siz are undefined).
- */
-int find_tape_device_and_size(char *dev, char *siz)
-{
-    char tmp[MAX_STR_LEN];
+ * @return the allocated tape string if success, NULL if failure
+ */
+char *mr_find_tape_device(void)
+{
+    char *tmp = NULL;
     char *command = NULL;
-    char cdr_exe[MAX_STR_LEN];
+    char *cdr_exe = NULL;
+    char *dev = NULL;
     int res;
 
     log_to_screen("I am looking for your tape streamer. Please wait.");
-    dev[0] = '\0';
-    if (siz != NULL) {
-        siz[0] = '\0';
-    }
     if (find_home_of_exe("cdrecord")) {
-        strcpy(cdr_exe, "cdrecord");
-    } else {
-        strcpy(cdr_exe, "dvdrecord");
+        mr_asprintf(&cdr_exe, "cdrecord");
+    } else {
+        mr_asprintf(&cdr_exe, "dvdrecord");
     }
     mr_asprintf(&command, "%s -scanbus 2> /dev/null | grep -i tape | wc -l", cdr_exe);
-    strcpy(tmp, call_program_and_get_last_line_of_output(command));
+    mr_asprintf(&tmp, "%s", call_program_and_get_last_line_of_output(command));
     mr_free(command);
 
     if (atoi(tmp) != 1) {
-        log_it
-            ("Either too few or too many tape streamers for me to detect...");
-        strcpy(dev, VANILLA_SCSI_TAPE);
-        return 1;
-    }
+        log_it("Either too few or too many tape streamers for me to detect...");
+        mr_asprintf(&dev, "%s", VANILLA_SCSI_TAPE);
+        mr_free(tmp);
+        return(dev);
+    }
+    mr_free(tmp);
+
     mr_asprintf(&command, "%s -scanbus 2> /dev/null | tr -s '\t' ' ' | grep \"[0-9]*,[0-9]*,[0-9]*\" | grep -v \"[0-9]*) \\*\" | grep -i TAPE | cut -d' ' -f2 | head -n1", cdr_exe);
-    strcpy(tmp, call_program_and_get_last_line_of_output(command));
+    mr_asprintf(&tmp, "%s", call_program_and_get_last_line_of_output(command));
     mr_free(command);
 
     if (strlen(tmp) < 2) {
         log_it("Could not find tape device");
-        return 1;
-    }
+        mr_free(tmp);
+        return(dev);
+    }
+    mr_free(tmp);
     mr_asprintf(&command, "%s -scanbus 2> /dev/null | tr -s '\t' ' ' | grep \"[0-9]*,[0-9]*,[0-9]*\" | grep -v \"[0-9]*) \\*\" | grep -i TAPE | cut -d' ' -f3 | cut -d')' -f1 | head -n1", cdr_exe);
-    strcpy(tmp, call_program_and_get_last_line_of_output(command));
+    mr_free(cdr_exe);
+
+    mr_asprintf(&tmp, "%s", call_program_and_get_last_line_of_output(command));
     mr_free(command);
 
-    strcpy(dev, VANILLA_SCSI_TAPE);
+    mr_asprintf(&dev, "%s", VANILLA_SCSI_TAPE);
     dev[strlen(dev) - 1] = '\0';
-    strcat(dev, tmp);           // e.g. '/dev/st0' becomes '/dev/stN'
+    mr_strcat(dev, tmp);            // e.g. '/dev/st0' becomes '/dev/stN'
+    mr_free(tmp);
+
     res = 0;
     if (!mt_says_tape_exists(dev)) {
-        strcpy(dev, ALT_TAPE);
+        mr_free(dev);
+        mr_asprintf(&dev, "%s", ALT_TAPE);
         if (!mt_says_tape_exists(dev)) {
             log_it("Cannot openin %s", dev);
-            strcpy(dev, "/dev/st0");
+            mr_free(dev);
+            mr_asprintf(&dev, "/dev/st0");
             if (!mt_says_tape_exists(dev)) {
                 log_it("Cannot openin %s", dev);
-                strcpy(dev, "/dev/osst0");
+                mr_free(dev);
+                mr_asprintf(&dev, "/dev/osst0");
                 if (!mt_says_tape_exists(dev)) {
                     res++;
@@ -318 +324 @@
     log_it("At this point, dev = %s and res = %d", dev, res);
 
-    strcpy(tmp, call_program_and_get_last_line_of_output("\
-cdrecord -scanbus 2> /dev/null | tr -s '\t' ' ' | \
-grep \"[0-9]*,[0-9]*,[0-9]*\" | grep -v \"[0-9]*) \\*\" | grep -i TAPE | \
-awk '{for(i=1; i<NF; i++) { if (index($i, \"GB\")>0) { print $i;};};};'"));
+    mr_asprintf(&tmp, call_program_and_get_last_line_of_output("cdrecord -scanbus 2> /dev/null | tr -s '\t' ' ' | grep \"[0-9]*,[0-9]*,[0-9]*\" | grep -v \"[0-9]*) \\*\" | grep -i TAPE | awk '{for(i=1; i<NF; i++) { if (index($i, \"GB\")>0) { print $i;};};};'"));
 
     if (mt_says_tape_exists(dev)) {
@@ -327 +330 @@
     } else {
         log_it("Turning %s", dev);
-        strcpy(tmp, (strrchr(dev, '/') != NULL) ? strrchr(dev, '/') : dev);
-        sprintf(dev, "/dev/os%s", tmp);
+        mr_free(tmp);
+        mr_asprintf(&tmp, (strrchr(dev, '/') != NULL) ? strrchr(dev, '/') : dev);
+        mr_free(dev);
+        mr_asprintf(&dev, "/dev/os%s", tmp);
         log_it("...into %s", dev);
         if (mt_says_tape_exists(dev)) {
@@ -340 +345 @@
 
     if (res) {
-        return (res);
+        mr_free(tmp);
+        return (dev);
     }
 
     if (strlen(tmp) < 2) {
         log_it("Warning - size of tape unknown");
-        return (0);
-    } else {
-        if (siz != NULL) {
-            strcpy(siz, tmp);
-        }
-        return (0);
-    }
-}
-
-
-
-
+    }
+    mr_free(tmp);
+    return (NULL);
+}