]> git.proxmox.com Git - pve-qemu-kvm.git/commitdiff
use g_strerror instead of strerror
authorDietmar Maurer <dietmar@proxmox.com>
Mon, 25 Feb 2013 06:40:19 +0000 (07:40 +0100)
committerDietmar Maurer <dietmar@proxmox.com>
Mon, 25 Feb 2013 06:40:19 +0000 (07:40 +0100)
Because strerror is not thread safe.

debian/patches/0004-introduce-new-vma-archive-format.patch
debian/patches/0005-add-regression-tests-for-backup.patch
debian/patches/0006-add-vm-state-to-backups.patch

index 0ff1c13183f6e443ccba913cb09c803afec9528b..4af6d6253eff48b4c97ea740a97853805b0e2b7c 100644 (file)
@@ -1,4 +1,4 @@
-From ec5c25c57d35ea662f70382a074620ef2abd0627 Mon Sep 17 00:00:00 2001
+From 6289a43696ca6f713a5d3bb9f95a5adb608a5e13 Mon Sep 17 00:00:00 2001
 From: Dietmar Maurer <dietmar@proxmox.com>
 Date: Tue, 13 Nov 2012 11:11:38 +0100
 Subject: [PATCH v5 4/6] introduce new vma archive format
@@ -114,7 +114,7 @@ index 0000000..9b715f2
 +
 diff --git a/vma-reader.c b/vma-reader.c
 new file mode 100644
-index 0000000..6d5a9ba
+index 0000000..bc36cba
 --- /dev/null
 +++ b/vma-reader.c
 @@ -0,0 +1,799 @@
@@ -351,7 +351,7 @@ index 0000000..6d5a9ba
 +    if (full_read(vmar->fd, vmar->head_data, sizeof(VmaHeader)) !=
 +        sizeof(VmaHeader)) {
 +        error_setg(errp, "can't read vma header - %s",
-+                   errno ? strerror(errno) : "got EOF");
++                   errno ? g_strerror(errno) : "got EOF");
 +        return -1;
 +    }
 +
@@ -375,7 +375,7 @@ index 0000000..6d5a9ba
 +    if (full_read(vmar->fd, vmar->head_data + sizeof(VmaHeader), need) !=
 +        need) {
 +        error_setg(errp, "can't read vma header data - %s",
-+                   errno ? strerror(errno) : "got EOF");
++                   errno ? g_strerror(errno) : "got EOF");
 +        return -1;
 +    }
 +
@@ -498,7 +498,7 @@ index 0000000..6d5a9ba
 +
 +    if (vmar->fd < 0) {
 +        error_setg(errp, "can't open file %s - %s\n", filename,
-+                   strerror(errno));
++                   g_strerror(errno));
 +        goto err;
 +    }
 +
@@ -816,7 +816,7 @@ index 0000000..6d5a9ba
 +    while (1) {
 +        int bytes = full_read(vmar->fd, buf + buf_pos, sizeof(buf) - buf_pos);
 +        if (bytes < 0) {
-+            error_setg(errp, "read failed - %s", strerror(errno));
++            error_setg(errp, "read failed - %s", g_strerror(errno));
 +            return -1;
 +        }
 +
@@ -919,7 +919,7 @@ index 0000000..6d5a9ba
 +
 diff --git a/vma-writer.c b/vma-writer.c
 new file mode 100644
-index 0000000..c292851
+index 0000000..9228ca6
 --- /dev/null
 +++ b/vma-writer.c
 @@ -0,0 +1,940 @@
@@ -1184,7 +1184,7 @@ index 0000000..c292851
 +                DPRINTF("vma_co_write restart %zd\n", done);
 +            } else {
 +                vma_writer_set_error(vmaw, "vma_co_write write error - %s",
-+                                     strerror(errno));
++                                     g_strerror(errno));
 +                done = -1; /* always return failure for partial writes */
 +                break;
 +            }
@@ -1311,7 +1311,7 @@ index 0000000..c292851
 +        vmaw->cmd = popen(p, "w");
 +        if (vmaw->cmd == NULL) {
 +            error_setg(errp, "can't popen command '%s' - %s\n", p,
-+                       strerror(errno));
++                       g_strerror(errno));
 +            goto err;
 +        }
 +        vmaw->fd = fileno(vmaw->cmd);
@@ -1346,7 +1346,7 @@ index 0000000..c292851
 +
 +        if (vmaw->fd < 0) {
 +            error_setg(errp, "can't open file %s - %s\n", filename,
-+                       strerror(errno));
++                       g_strerror(errno));
 +            goto err;
 +        }
 +    }
@@ -1741,12 +1741,12 @@ index 0000000..c292851
 +    if (vmaw->cmd) {
 +        if (pclose(vmaw->cmd) < 0) {
 +            vma_writer_set_error(vmaw, "vma_writer_close: "
-+                                 "pclose failed - %s", strerror(errno));
++                                 "pclose failed - %s", g_strerror(errno));
 +        }
 +    } else {
 +        if (close(vmaw->fd) < 0) {
 +            vma_writer_set_error(vmaw, "vma_writer_close: "
-+                                 "close failed - %s", strerror(errno));
++                                 "close failed - %s", g_strerror(errno));
 +        }
 +    }
 +
@@ -1865,7 +1865,7 @@ index 0000000..c292851
 +
 diff --git a/vma.c b/vma.c
 new file mode 100644
-index 0000000..d3fe39c
+index 0000000..6633aa5
 --- /dev/null
 +++ b/vma.c
 @@ -0,0 +1,561 @@
@@ -2053,7 +2053,7 @@ index 0000000..d3fe39c
 +
 +    if (mkdir(dirname, 0777) < 0) {
 +        g_error("unable to create target directory %s - %s",
-+                dirname, strerror(errno));
++                dirname, g_strerror(errno));
 +    }
 +
 +    GList *l = vma_reader_get_config_data(vmar);
@@ -2075,7 +2075,7 @@ index 0000000..d3fe39c
 +
 +        FILE *map = fopen(readmap, "r");
 +        if (!map) {
-+            g_error("unable to open fifo %s - %s", readmap, strerror(errno));
++            g_error("unable to open fifo %s - %s", readmap, g_strerror(errno));
 +        }
 +
 +        while (1) {
@@ -2133,7 +2133,7 @@ index 0000000..d3fe39c
 +            vmstate_fd = open(statefn, O_WRONLY|O_CREAT|O_EXCL, 0644);
 +            if (vmstate_fd < 0) {
 +                g_error("create vmstate file '%s' failed - %s", statefn,
-+                        strerror(errno));
++                        g_strerror(errno));
 +            }
 +            g_free(statefn);
 +        } else if (di) {
@@ -2195,7 +2195,7 @@ index 0000000..d3fe39c
 +                                           dirname, di->devname);
 +                if (rename(tmpfn, fn) != 0) {
 +                    g_error("rename %s to %s failed - %s",
-+                            tmpfn, fn, strerror(errno));
++                            tmpfn, fn, g_strerror(errno));
 +                }
 +            }
 +        }
index 83d55fac30cd0413e33b1c55beb3859578baaa77..fc0679af6b1863aa6f2fc17bb7d61d8061f63c8c 100644 (file)
@@ -1,4 +1,4 @@
-From 0c613da24e8de22960e5a9f0f1636d9e2b0bd18d Mon Sep 17 00:00:00 2001
+From 348a009b2c63ac4d9b6dad7fe2169272c46221f2 Mon Sep 17 00:00:00 2001
 From: Dietmar Maurer <dietmar@proxmox.com>
 Date: Wed, 14 Nov 2012 09:57:04 +0100
 Subject: [PATCH v5 5/6] add regression tests for backup
@@ -52,7 +52,7 @@ index 567e36e..136be84 100644
  -include $(wildcard tests/*.d)
 diff --git a/tests/backup-test.c b/tests/backup-test.c
 new file mode 100644
-index 0000000..039ac1d
+index 0000000..47a9664
 --- /dev/null
 +++ b/tests/backup-test.c
 @@ -0,0 +1,529 @@
@@ -373,7 +373,7 @@ index 0000000..039ac1d
 +        }
 +        if (res != sizeof(buf)) {
 +            g_error("can't initialize file %s - %s %d\n",
-+                    filename, strerror(errno), res);
++                    filename, g_strerror(errno), res);
 +        }
 +    }
 +
index dfed9151fc096af112f53b0a44205aaa3d499b45..d68b4270d1661cdb73d84de51c537b26a80d42f2 100644 (file)
@@ -1,4 +1,4 @@
-From aa580be154923056a52fdeec429f7e80f04b456e Mon Sep 17 00:00:00 2001
+From c209554171466e2aa8b1eca92d952429faca5fc6 Mon Sep 17 00:00:00 2001
 From: Dietmar Maurer <dietmar@proxmox.com>
 Date: Thu, 29 Nov 2012 10:46:49 +0100
 Subject: [PATCH v5 6/6] add vm state to backups