varun-r-mallya/sysprof
1
0
Fork 0
mirror of https://github.com/varun-r-mallya/sysprof.git synced 2025-12-31 20:36:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

libsysprof-capture: Use mkostemp() rather than g_file_open_tmp()

Browse Source 
Signed-off-by: Philip Withnall <withnall@endlessm.com>

Helps: #40
This commit is contained in:
Philip Withnall
2020-07-02 12:23:28 +01:00
parent 43530e3613
commit 621e7ea314
1 changed files with 43 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

48
src/libsysprof-capture/sysprof-platform.c
View File

@ -58,12 +58,34 @@
#include <glib.h> #include <glib.h>
#include <glib/gstdio.h> #include <glib/gstdio.h>
#include <stdlib.h>
#include <sys/syscall.h> #include <sys/syscall.h>
#include <unistd.h> #include <unistd.h>
#include "sysprof-capture-util-private.h" #include "sysprof-capture-util-private.h"
#include "sysprof-platform.h" #include "sysprof-platform.h"
#ifndef __NR_memfd_create
static const char *
get_tmpdir (void)
{
const char *tmpdir = NULL;
if (tmpdir == NULL || *tmpdir == '\0')
tmpdir = getenv ("TMPDIR");
#ifdef P_tmpdir
if (tmpdir == NULL || *tmpdir == '\0')
tmpdir = P_tmpdir;
#endif
if (tmpdir == NULL || *tmpdir == '\0')
tmpdir = "/tmp";
return tmpdir;
}
#endif /* !__NR_memfd_create */
/** /**
* sysprof_memfd_create: * sysprof_memfd_create:
* @name: (nullable): A descriptive name for the memfd or %NULL * @name: (nullable): A descriptive name for the memfd or %NULL
@ -82,8 +104,10 @@ sysprof_memfd_create (const char *name)
name = "[sysprof]"; name = "[sysprof]";
return syscall (__NR_memfd_create, name, 0); return syscall (__NR_memfd_create, name, 0);
#else #else
gchar *name_used = NULL;
int fd; int fd;
const char *tmpdir;
char *template = NULL;
size_t template_len = 0;
/* /*
* TODO: It would be nice to ensure tmpfs * TODO: It would be nice to ensure tmpfs
@ -93,14 +117,28 @@ sysprof_memfd_create (const char *name)
* that the tmpfile we open is on tmpfs so that we get anonymous backed * that the tmpfile we open is on tmpfs so that we get anonymous backed
* pages after unlinking. * pages after unlinking.
*/ */
fd = g_file_open_tmp (NULL, &name_used, NULL);
if (name_used != NULL) tmpdir = get_tmpdir ();
template_len = strlen (tmpdir) + 1 + strlen ("sysprof-XXXXXX") + 1;
template = sysprof_malloc0 (template_len);
if (template == NULL)
{ {
g_unlink (name_used); errno = ENOMEM;
g_free (name_used); return -1;
} }
snprintf (template, template_len, "%s/sysprof-XXXXXX", tmpdir);
fd = TEMP_FAILURE_RETRY (mkostemp (template, O_BINARY | O_CLOEXEC));
if (fd < 0)
{
free (template);
return -1;
}
unlink (template);
free (template);
return fd; return fd;
#endif #endif
} }