From patchwork Thu Aug 31 20:34:16 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Adhemerval Zanella X-Patchwork-Id: 111443 Delivered-To: patch@linaro.org Received: by 10.140.95.112 with SMTP id h103csp30103qge; Thu, 31 Aug 2017 13:34:34 -0700 (PDT) X-Received: by 10.99.1.20 with SMTP id 20mr3906594pgb.445.1504211674899; Thu, 31 Aug 2017 13:34:34 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1504211674; cv=none; d=google.com; s=arc-20160816; b=jk852OUHFp+LrVEaoMZ+qBLcgcBgie1xPTPvoGKexlhx7INEil+T2YLmup78pzyeLd TUi3dIwHBTwW/Hdi/X0li4xPEYAOaKxb2+Jtc7NAHKufSA9izQCm/SLTmah8H06QK1pg L9ItpWQldMnmwmRk5OsWvuA3yS59XjE3ApaKJcegDM6LGmjpvFscKjOTEvrMgDCE+2Bh QX4+DNz1OcnozeqiUINbY/bbKTxlFsQxv0W6N9wColgkc4cVKlLYwtifxGzLUnDOhqWd H6Nk0xakTbbU0bsPbumcneP/KLeFi9KMalnHEoEp4XySTsSnYGgFoSptSgQmwOIcTt70 eTfA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=message-id:date:subject:to:from:delivered-to:sender:list-help :list-post:list-archive:list-subscribe:list-unsubscribe:list-id :precedence:mailing-list:dkim-signature:domainkey-signature :arc-authentication-results; bh=TRM1kyMcyf9+MfNMeRD2nZvei9tmk2X+EkOEsFaC/4Y=; b=ihwWq3KTgD1S5s8erlXTkQSm1mnEXaHhk1s2bZgSYZS+YnWO6l3aADFx5o50yw6/3E 6RgPM6ShofIMwE400GOsP5JYtUu5+BV5IFsJAlx65AYl4DTgh4ElmC7Q1SOMCCqt0W1I EJeew7aZXs3jTJtj/gGSYd+m62f4Yg8fMNHi27JTAq02Mjbb5j3y7h87SvrvkmnAjWr9 k5ifvojRf5HQxC/h+7ReW5FsxnFu0dpXteAdwKE573CFaW0xfe00OoBYKW4VTIVvUVMv qeKeQugfQvB7f043uv+C0zIA3OKyermeVMZRIFC/tcoCwdxc2j796GvDnB40s1KY52yA Agiw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@sourceware.org header.s=default header.b=mKFFbOr+; spf=pass (google.com: domain of libc-alpha-return-83999-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) smtp.mailfrom=libc-alpha-return-83999-patch=linaro.org@sourceware.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from sourceware.org (server1.sourceware.org. [209.132.180.131]) by mx.google.com with ESMTPS id n22si418057pfj.334.2017.08.31.13.34.34 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 31 Aug 2017 13:34:34 -0700 (PDT) Received-SPF: pass (google.com: domain of libc-alpha-return-83999-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) client-ip=209.132.180.131; Authentication-Results: mx.google.com; dkim=pass header.i=@sourceware.org header.s=default header.b=mKFFbOr+; spf=pass (google.com: domain of libc-alpha-return-83999-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) smtp.mailfrom=libc-alpha-return-83999-patch=linaro.org@sourceware.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:from:to:subject:date:message-id; q=dns; s= default; b=eLNH1MX3QChqCJeVgnNTpjHNhkTsvQnOQqV4NHgLxrAS+I0TT4p2Y 3aJIU9gVCTqdVrumKROYCZ0tKBZGMwVWi7j0OSX+jzaOLfv6Rf5vjPwY5dOHPJwk 9Iklh7FfM+uZrUZpnnt2U6zmV9DeU1QQvvabYak+v5ge7DQo9SnheQ= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:from:to:subject:date:message-id; s=default; bh=u29+XIqlO7VH5JRm28nM62Gx7/c=; b=mKFFbOr+Jfwa3kIr3sBylz/P9kmK 0T4726H6qfluBu5Lc4AZImFklgeaTPscoZPP6L6hhDPuJ2GQcNy0BbPWGE0pD0Sb dniXN8k2vniyRL/pfKIsLy4Ylauw8ZRNc2lk70xG2hlOhcvg16lwFz2+1A+05L4g eVEg04D/x4gTSeM= Received: (qmail 60030 invoked by alias); 31 Aug 2017 20:34:25 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 60020 invoked by uid 89); 31 Aug 2017 20:34:25 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-26.4 required=5.0 tests=BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE, RCVD_IN_SORBS_SPAM, SPF_PASS autolearn=ham version=3.3.2 spammy= X-HELO: mail-qt0-f173.google.com X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:date:message-id; bh=TRM1kyMcyf9+MfNMeRD2nZvei9tmk2X+EkOEsFaC/4Y=; b=tMfx+ccvxngsRkSL67WBnyfAe8rVOVoGdZlZiHtIQB7GMGfOtvaU6SxkvCu5t2VaIC X/enObdpqWRKcX+ctY5gp9na+8Qbxb+kfcOBfqoeKAhoXo3dTorKP2jUOk1b79TWipc4 YHuF7dsuYjUtN3hH+Y0muBzqU62Xp0MgN/ttRDa8uJvagLGjbpd6YjtcgDkgKutP1H0a ZC7JA3QEAZneAzwiqvQuqz1boxYg05vmHzvBimLK6CAD5fGzWfncqFs254jZjik8RbaX aG0A0i7PmwAupL+sfHdbvFYwMGo0UbuwCglby1qFRR+AFAjZFzxY49YMBFkrZe4WoxRa xxjg== X-Gm-Message-State: AHYfb5hFGNe3BMNCOE8GBVyGgTrgoCMZLWpF4lE2ZYzald/moYxmGLzu j0XBU4oaCgK2RMRzJEIs6Q== X-Google-Smtp-Source: ADKCNb4nMgzMWGOFqYnaeRNO1Wml6LWcCOwnle7VkqXATXqNjypQim9+xnMh7hrTcNE07wy4e0zW7Q== X-Received: by 10.237.51.35 with SMTP id u32mr9059829qtd.207.1504211661559; Thu, 31 Aug 2017 13:34:21 -0700 (PDT) From: Adhemerval Zanella To: libc-alpha@sourceware.org Subject: [PATCH] linux: Implement tmpfile with O_TMPFILE (BZ#21530) Date: Thu, 31 Aug 2017 17:34:16 -0300 Message-Id: <1504211656-9263-1-git-send-email-adhemerval.zanella@linaro.org> This patch adds support to use O_TMPFILE on tmpfile on Linux. This is similar previous suggestion by Andreas Schwab [1] with the difference the file descriptor creation is parameterized to compartmentalize Linux only open flags (O_TMPFILE) on sysdep folder. Checked on x86_64-linux-gnu. Adhemerval Zanella Andreas Schwab [BZ #21530] * include/stdio.h (__gen_tempfd): New function. * stdio-common/Makefile (routines): Add gentempfd. * stdio-common/gentempfd.c: New file. * sysdeps/unix/sysv/linux/gentempfd.c: Likewise. * stdio-common/tmpfile.c (tmpfile): First try to use a system specific unnamed file first. [1] https://sourceware.org/ml/libc-alpha/2017-06/msg01293.html --- ChangeLog | 11 +++++++++++ include/stdio.h | 3 +++ stdio-common/Makefile | 2 +- stdio-common/gentempfd.c | 26 ++++++++++++++++++++++++++ stdio-common/tmpfile.c | 29 ++++++++++++++++++++--------- sysdeps/unix/sysv/linux/gentempfd.c | 34 ++++++++++++++++++++++++++++++++++ 6 files changed, 95 insertions(+), 10 deletions(-) create mode 100644 stdio-common/gentempfd.c create mode 100644 sysdeps/unix/sysv/linux/gentempfd.c -- 2.7.4 diff --git a/include/stdio.h b/include/stdio.h index 509447c..87e0e10 100644 --- a/include/stdio.h +++ b/include/stdio.h @@ -191,5 +191,8 @@ libc_hidden_proto (__obstack_vprintf_chk) extern FILE * __fmemopen (void *buf, size_t len, const char *mode); libc_hidden_proto (__fmemopen) +extern int __gen_tempfd (int flags); +libc_hidden_proto (__gen_tempfd) + # endif /* not _ISOMAC */ #endif /* stdio.h */ diff --git a/stdio-common/Makefile b/stdio-common/Makefile index 397e0c2..2c3c2e5 100644 --- a/stdio-common/Makefile +++ b/stdio-common/Makefile @@ -39,7 +39,7 @@ routines := \ flockfile ftrylockfile funlockfile \ isoc99_scanf isoc99_vscanf isoc99_fscanf isoc99_vfscanf isoc99_sscanf \ isoc99_vsscanf \ - psiginfo + psiginfo gentempfd aux := errlist siglist printf-parsemb printf-parsewc fxprintf diff --git a/stdio-common/gentempfd.c b/stdio-common/gentempfd.c new file mode 100644 index 0000000..d40c57d --- /dev/null +++ b/stdio-common/gentempfd.c @@ -0,0 +1,26 @@ +/* Generate a temporary file descriptor. Generic/POSIX version. + Copyright (C) 2017 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include + +int +__gen_tempfd (int flags) +{ + return -1; +} +libc_hidden_def (__gen_tempfd) diff --git a/stdio-common/tmpfile.c b/stdio-common/tmpfile.c index e6030be..3e35345 100644 --- a/stdio-common/tmpfile.c +++ b/stdio-common/tmpfile.c @@ -34,23 +34,34 @@ FILE * tmpfile (void) { - char buf[FILENAME_MAX]; int fd; FILE *f; - - if (__path_search (buf, FILENAME_MAX, NULL, "tmpf", 0)) - return NULL; int flags = 0; #ifdef FLAGS flags = FLAGS; #endif - fd = __gen_tempname (buf, 0, flags, __GT_FILE); + + /* First try a system specific method. */ + fd = __gen_tempfd (flags); + if (fd < 0) - return NULL; + { + char buf[FILENAME_MAX]; - /* Note that this relies on the Unix semantics that - a file is not really removed until it is closed. */ - (void) __unlink (buf); + if (__path_search (buf, sizeof buf, NULL, "tmpf", 0)) + return NULL; + + fd = __gen_tempname (buf, 0, flags, __GT_FILE); + if (fd < 0) + return NULL; + + /* Note that this relies on the Unix semantics that + a file is not really removed until it is closed. */ + (void) __unlink (buf); + } + + if (fd < 0) + return NULL; if ((f = __fdopen (fd, "w+b")) == NULL) __close (fd); diff --git a/sysdeps/unix/sysv/linux/gentempfd.c b/sysdeps/unix/sysv/linux/gentempfd.c new file mode 100644 index 0000000..902cbe2 --- /dev/null +++ b/sysdeps/unix/sysv/linux/gentempfd.c @@ -0,0 +1,34 @@ +/* Generate a temporary file descriptor. Linux version. + Copyright (C) 2017 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include +#include +#include + +int +__gen_tempfd (int flags) +{ + int fd = __open (P_tmpdir, O_RDWR | O_TMPFILE | O_EXCL | flags, + S_IRUSR | S_IWUSR); + if (fd < 0 && errno == ENOENT && strcmp (P_tmpdir, "/tmp") != 0) + fd = __open ("/tmp", O_RDWR | O_TMPFILE | O_EXCL | flags, + S_IRUSR | S_IWUSR); + + return fd; +} +libc_hidden_def (__gen_tempfd)