[devel] I: temporary file handling: C coding guideline

[devel] I: temporary file handling: C coding guideline

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 779 bytes --]

Greetings!

Я планирую добавить "C coding guideline" в дополнение к нынешнему shell
coding guideline из SPP:

#define _GNU_SOURCE

#include <stdio.h>
#include <error.h>
#include <errno.h>
#include <stdlib.h>
#include <unistd.h>

...
char *fname;
int fd;
FILE *fp;

if (asprintf (&fname, "%s/progname.XXXXXX", getenv ("TMPDIR") ?: P_tmpdir) < 0)
	error (EXIT_FAILURE, errno, "asprintf");

if ((fd = mkstemp (fname)) < 0)
	error (EXIT_FAILURE, errno, "mkstemp: %s", fname);

if (unlink (fname) < 0)
	error (EXIT_FAILURE, errno, "unlink: %s", fname);

if ((fp = fdopen (fd, "w+")) == NULL)
	error (EXIT_FAILURE, errno, "fdopen: %s", fname);

...

if (fclose (fp))
	error (EXIT_FAILURE, errno, "fclose: %s", fname);

free (fname);
fname = NULL;

Комментарии приветствуются.


--
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] I: temporary file handling: C coding guideline

[Stanislav Ievlev]

On Tue, Jun 03, 2003 at 12:17:29PM +0400, Dmitry V. Levin wrote:
> Greetings!
> 
> Я планирую добавить "C coding guideline" в дополнение к нынешнему shell
> coding guideline из SPP:
> 
> #define _GNU_SOURCE
> 
> #include <stdio.h>
> #include <error.h>
> #include <errno.h>
> #include <stdlib.h>
> #include <unistd.h>
> 
> ...
> char *fname;
> int fd;
> FILE *fp;
> 
> if (asprintf (&fname, "%s/progname.XXXXXX", getenv ("TMPDIR") ?: P_tmpdir) < 0)
> 	error (EXIT_FAILURE, errno, "asprintf");
> 
> if ((fd = mkstemp (fname)) < 0)
> 	error (EXIT_FAILURE, errno, "mkstemp: %s", fname);
> 
> if (unlink (fname) < 0)
> 	error (EXIT_FAILURE, errno, "unlink: %s", fname);
> 
> if ((fp = fdopen (fd, "w+")) == NULL)
> 	error (EXIT_FAILURE, errno, "fdopen: %s", fname);
> 
> ...
> 
> if (fclose (fp))
> 	error (EXIT_FAILURE, errno, "fclose: %s", fname);
> 
> free (fname);
> fname = NULL;
> 
> Комментарии приветствуются.
Хочется man-page по asprintf.
Я тут не очень понял про коды возврата.
0 это тоже будет ошибкой в данном случае?

Ещё надо бы наверное portable вариант без gcc-змов. Это потребуется всем
мантейнерам которые захотят убедить авторов в своём патче.
> 
> 
> --
> ldv

[devel] temporary file handling: questions

[Grigory Batalov]

On Tue, 3 Jun 2003 12:17:29 +0400
"Dmitry V. Levin" <ldv@altlinux.org> wrote:

> #define _GNU_SOURCE
> 
> #include <stdio.h>
> #include <error.h>
> #include <errno.h>
> #include <stdlib.h>
> #include <unistd.h>
> 
> ...
> char *fname;
> int fd;
> FILE *fp;
> 
> if (asprintf (&fname, "%s/progname.XXXXXX", getenv ("TMPDIR") ?: P_tmpdir) < 0)
> 	error (EXIT_FAILURE, errno, "asprintf");
> 
> if ((fd = mkstemp (fname)) < 0)
> 	error (EXIT_FAILURE, errno, "mkstemp: %s", fname);
> 
> if (unlink (fname) < 0)
> 	error (EXIT_FAILURE, errno, "unlink: %s", fname);
> 
> if ((fp = fdopen (fd, "w+")) == NULL)
> 	error (EXIT_FAILURE, errno, "fdopen: %s", fname);
> 
> ...
> 
> if (fclose (fp))
> 	error (EXIT_FAILURE, errno, "fclose: %s", fname);
> 
> free (fname);
> fname = NULL;
> 
> Комментарии приветствуются.

  Если я правильно понимаю, fdopen/fclose используются, чтобы
  "занять" файловый дескриптор потоком и предотвратить
  немедленное удаление функцией unlink?

  Или fdopen/fclose приведены для примера потокового
  ввода/вывода, и их можно опустить (например, мне достаточно
  open/close)? В этом случае "оттягивать до последнего"
  нужно вызов close(fd) ?

Re: [devel] temporary file handling: questions

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 914 bytes --]

On Wed, Jun 11, 2003 at 03:14:59AM +0400, Grigory Batalov wrote:
>   Если я правильно понимаю, fdopen/fclose используются, чтобы
>   "занять" файловый дескриптор потоком и предотвратить
>   немедленное удаление функцией unlink?

Нет, неправильно.
fdopen(3) использует передаваемый в качестве аргумента дескриптор.

>   Или fdopen/fclose приведены для примера потокового
>   ввода/вывода, и их можно опустить (например, мне достаточно
>   open/close)? В этом случае "оттягивать до последнего"
>   нужно вызов close(fd) ?

Да, fdopen(3) приведён в качестве (реального) примера в случае, когда
осуществляется чтение/запись во временный файл.  Можно не использовать
stdio и обойтись read(2)/write(2); в этом случае вместо fclose(3) будет
вызов close(2).

Главное - сделать unlink(2) как можно раньше, и обязательно до закрытия
дескриптора файла.  Следует избегать файловых операций с именем временного
файла.


--
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] temporary file handling: questions

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 394 bytes --]

* Dmitry V. Levin <ldv@altlinux.org> [030611 03:33]:
> Главное - сделать unlink(2) как можно раньше, и обязательно до
> закрытия дескриптора файла.  Следует избегать файловых операций
> с именем временного файла.
Хм... А если этот временный файл (и не один) надо передать другой
программе?

-- 
Regards, Sir Raorn.
-------------------
inn - это несравнимо хуже, чем wu-ftpd.
		-- ldv in devel@

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] temporary file handling: questions

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 634 bytes --]

On Wed, Jun 11, 2003 at 11:31:01AM +0400, Alexey I. Froloff wrote:
> * Dmitry V. Levin <ldv@altlinux.org> [030611 03:33]:
> > Главное - сделать unlink(2) как можно раньше, и обязательно до
> > закрытия дескриптора файла.  Следует избегать файловых операций
> > с именем временного файла.
> Хм... А если этот временный файл (и не один) надо передать другой
> программе?

Все знают, что это, вообще говоря, плохо.
На эту тему статьи написаны, можно почитать.

Впрочем, безопасное создание временного файла и последующее его безопасное
использование - это два разных вопроса, и многие просто ограничиваются
решением первого. :(


--
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] temporary file handling: questions

[Alexey I. Froloff]

[-- Attachment #1: Type: text/plain, Size: 611 bytes --]

On Wed, Jun 11, 2003 at 12:19:20PM +0400, Dmitry V. Levin wrote:
> > > Главное - сделать unlink(2) как можно раньше, и обязательно до
> > > закрытия дескриптора файла.  Следует избегать файловых операций
> > > с именем временного файла.
> > Хм... А если этот временный файл (и не один) надо передать другой
> > программе?
> Все знают, что это, вообще говоря, плохо.
> На эту тему статьи написаны, можно почитать.
Стоп! Возьмём например rpm. Создаётся временный файл потом он
"скармливается" шеллу. Как это сделать "хорошо"?

И почему "хорошо" - использовать mktemp(1) в shell-скриптах?

-- 
Regards,
Sir Raorn.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] temporary file handling: questions

[Stanislav Ievlev]

On Wed, Jun 11, 2003 at 11:31:01AM +0400, Alexey I. Froloff wrote:
> * Dmitry V. Levin <ldv@altlinux.org> [030611 03:33]:
> > Главное - сделать unlink(2) как можно раньше, и обязательно до
> > закрытия дескриптора файла.  Следует избегать файловых операций
> > с именем временного файла.
> Хм... А если этот временный файл (и не один) надо передать другой
> программе?
А зачем это делать?
Разве нет других способов передачи данных между процессами?
> 
> -- 
> Regards, Sir Raorn.
> -------------------
> inn - это несравнимо хуже, чем wu-ftpd.
> 		-- ldv in devel@

[devel] Re: temporary file handling: questions

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 423 bytes --]

On Wed, Jun 11, 2003 at 03:14:59AM +0400, Grigory Batalov wrote:
>   Если я правильно понимаю, fdopen/fclose используются, чтобы
>   "занять" файловый дескриптор потоком и предотвратить
>   немедленное удаление функцией unlink?

Нет, файлы -- сами по себе, дескрипторы -- сами по себе и stdio потоки --
тоже сами по себе.  У меня несколько месяцев назад от этого крыша
съезжала.  А теперь вроде так и надо.  Как же ещё? :)

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]