Hash :
524d6a2e
Author :
Date :
2009-12-31T11:53:33
stdio: warn on suspicious uses Using gets is almost ALWAYS wrong (it is extremely rare that you have full control over stdin). POSIX 2008 marked it as obsolete, even though C89 requires it. Attach a warning to remind developers. Add a comment to sprintf explaining why it does not get this treatment. Improve the warnings for fseek/fseeko (and ftell/ftello), with comments justifying our position. Some of our unit tests never use large files, so rather than drag in a dependency on fseeko, they should be the first compilation units to use _GL_NO_LARGE_FILES. * modules/stdio (Depends-on): Add warn-on-use. (Makefile.am): Provide new substitutions. * m4/stdio_h.m4 (gl_STDIO_H): Check for inline, ftello, and fseeko. * lib/stdio.in.h (gets): Always warn on use. (fseek, ftell): Adjust when warnings are issued, and honor _GL_NO_LARGE_FILES as a way to silence the warning. * tests/test-fpurge.c [!GNULIB_FSEEK]: Use new means to squelch any warning about large file offsets. * tests/test-freadable.c [!GNULIB_FSEEK]: Likewise. * tests/test-freading.c [!GNULIB_FSEEK]: Likewise. * tests/test-fseeko.c [!GNULIB_FSEEK]: Likewise. * tests/test-ftell.c [!GNULIB_FSEEK]: Likewise. * tests/test-ftello.c [!GNULIB_FSEEK]: Likewise. * tests/test-fwritable.c [!GNULIB_FSEEK]: Likewise. * tests/test-fwriting.c [!GNULIB_FSEEK]: Likewise. * tests/test-getopt.c [!GNULIB_FTELL]: Likewise. Signed-off-by: Eric Blake <ebb9@byu.net>
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118
/* Test of ftello() function.
Copyright (C) 2007, 2008, 2009, 2010 Free Software Foundation, Inc.
This program is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation; either version 3 of the License, or
(at your option) any later version.
This program 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 General Public License for more details.
You should have received a copy of the GNU General Public License
along with this program. If not, see <http://www.gnu.org/licenses/>. */
/* Written by Bruno Haible <bruno@clisp.org>, 2007. */
#include <config.h>
/* None of the files accessed by this test are large, so disable the
fseek link warning if we are not using the gnulib fseek module. */
#define _GL_NO_LARGE_FILES
#include <stdio.h>
#include "signature.h"
SIGNATURE_CHECK (ftello, off_t, (FILE *));
#include "binary-io.h"
#include "macros.h"
#ifndef FUNC_UNGETC_BROKEN
# define FUNC_UNGETC_BROKEN 0
#endif
int
main (int argc, char **argv _GL_UNUSED)
{
int ch;
/* Assume stdin is seekable iff argc > 1. */
if (argc == 1)
{
ASSERT (ftell (stdin) == -1);
ASSERT (ftello (stdin) == -1);
return 0;
}
/* mingw ftell is unreliable on text mode input. */
SET_BINARY (0);
/* Simple tests. For each test, make sure ftell and ftello agree. */
ASSERT (ftell (stdin) == 0);
ASSERT (ftello (stdin) == 0);
ch = fgetc (stdin);
ASSERT (ch == '#');
ASSERT (ftell (stdin) == 1);
ASSERT (ftello (stdin) == 1);
/* Test ftell after ungetc of read input. */
ch = ungetc ('#', stdin);
ASSERT (ch == '#');
ASSERT (ftell (stdin) == 0);
ASSERT (ftello (stdin) == 0);
ch = fgetc (stdin);
ASSERT (ch == '#');
ASSERT (ftell (stdin) == 1);
ASSERT (ftello (stdin) == 1);
/* Test ftell after fseek. */
ASSERT (fseek (stdin, 2, SEEK_SET) == 0);
ASSERT (ftell (stdin) == 2);
ASSERT (ftello (stdin) == 2);
/* Test ftell after random ungetc. */
ch = fgetc (stdin);
ASSERT (ch == '/');
ch = ungetc ('@', stdin);
ASSERT (ch == '@');
ASSERT (ftell (stdin) == 2);
ASSERT (ftello (stdin) == 2);
ch = fgetc (stdin);
ASSERT (ch == '@');
ASSERT (ftell (stdin) == 3);
ASSERT (ftello (stdin) == 3);
if (2 < argc)
{
if (FUNC_UNGETC_BROKEN)
{
fputs ("Skipping test: ungetc cannot handle arbitrary bytes\n",
stderr);
return 77;
}
/* Test ftell after ungetc without read. */
ASSERT (fseek (stdin, 0, SEEK_CUR) == 0);
ASSERT (ftell (stdin) == 3);
ASSERT (ftello (stdin) == 3);
ch = ungetc ('~', stdin);
ASSERT (ch == '~');
ASSERT (ftell (stdin) == 2);
ASSERT (ftello (stdin) == 2);
}
#if !defined __MINT__ /* FreeMiNT has problems seeking past end of file */
/* Test ftell beyond end of file. */
ASSERT (fseek (stdin, 0, SEEK_END) == 0);
ch = ftello (stdin);
ASSERT (fseek (stdin, 10, SEEK_END) == 0);
ASSERT (ftell (stdin) == ch + 10);
ASSERT (ftello (stdin) == ch + 10);
#endif
return 0;
}