[Libguestfs] [PATCH 6/6] lib: Use guestfs_int_make_temp_path in a few more places.
Pino Toscano
ptoscano at redhat.com
Wed Sep 20 15:28:35 UTC 2017
On Tuesday, 19 September 2017 13:38:27 CEST Richard W.M. Jones wrote:
> diff --git a/lib/command.c b/lib/command.c
> index bc469de59..7018c3ac0 100644
> --- a/lib/command.c
> +++ b/lib/command.c
> @@ -815,8 +815,9 @@ guestfs_int_cmd_pipe_run (struct command *cmd, const char *mode)
> if (guestfs_int_lazy_make_tmpdir (cmd->g) == -1)
> goto error;
>
> - cmd->error_file =
> - safe_asprintf (cmd->g, "%s/cmderr.%d", cmd->g->tmpdir, ++cmd->g->unique);
> + cmd->error_file = guestfs_int_make_temp_path (cmd->g, "cmderr", "txt");
> + if (!cmd->error_file)
> + goto error;
> errfd = open (cmd->error_file,
> O_WRONLY|O_CREAT|O_NOCTTY|O_TRUNC|O_CLOEXEC, 0600);
> if (errfd == -1) {
> diff --git a/lib/drives.c b/lib/drives.c
> index 117c8bf85..f43e64b10 100644
> --- a/lib/drives.c
> +++ b/lib/drives.c
> @@ -989,7 +989,9 @@ guestfs_impl_add_drive_scratch (guestfs_h *g, int64_t size,
> */
> if (guestfs_int_lazy_make_tmpdir (g) == -1)
> return -1;
> - filename = safe_asprintf (g, "%s/scratch.%d", g->tmpdir, ++g->unique);
> + filename = guestfs_int_make_temp_path (g, "scratch", "img");
> + if (!filename)
> + return -1;
>
> /* Create a raw format temporary disk. */
> if (guestfs_disk_create (g, filename, "raw", size, -1) == -1)
The calls to guestfs_int_lazy_make_tmpdir can be removed in the files
above, like done already in the launch-*.c files.
--
Pino Toscano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20170920/2421f6e9/attachment.sig>
More information about the Libguestfs
mailing list