aboutsummaryrefslogtreecommitdiff
path: root/libio
diff options
context:
space:
mode:
authorSiddhesh Poyarekar <siddhesh@redhat.com>2014-03-10 16:20:01 +0530
committerSiddhesh Poyarekar <siddhesh@redhat.com>2014-03-17 19:44:51 +0530
commitb1dbb426e164ad1236c2c76268e03fec5c7a7bbe (patch)
tree9e3a263e48066fd376234a4e03cebc542eb6b36a /libio
parentfcd89ebe4f5ea948ff4c796771b918cde8960721 (diff)
downloadglibc-b1dbb426e164ad1236c2c76268e03fec5c7a7bbe.zip
glibc-b1dbb426e164ad1236c2c76268e03fec5c7a7bbe.tar.gz
glibc-b1dbb426e164ad1236c2c76268e03fec5c7a7bbe.tar.bz2
Fix up return codes for tests in tst-ftell-active-handler
The test functions used a variable ret to store failure codes for individual tests, but the variable was incorrectly used to record other failure codes too, resulting in overwriting of the tests status. This is now fixed by making sure that the ret variable is used only for recording test failures. * libio/tst-ftell-active-handler.c (do_ftell_test): Don't mix up test status with function return status. (do_write_test): Likewise. (do_append_test): Likewise.
Diffstat (limited to 'libio')
-rw-r--r--libio/tst-ftell-active-handler.c49
1 files changed, 31 insertions, 18 deletions
diff --git a/libio/tst-ftell-active-handler.c b/libio/tst-ftell-active-handler.c
index 54bfe63..5d5fc26 100644
--- a/libio/tst-ftell-active-handler.c
+++ b/libio/tst-ftell-active-handler.c
@@ -119,17 +119,20 @@ do_ftell_test (const char *filename)
{
FILE *fp;
int fd;
+ int fileret;
+
printf ("\tftell: %s (file, \"%s\"): ", j == 0 ? "fdopen" : "fopen",
test_modes[i].mode);
if (j == 0)
- ret = get_handles_fdopen (filename, fd, fp, test_modes[i].fd_mode,
- test_modes[i].mode);
+ fileret = get_handles_fdopen (filename, fd, fp,
+ test_modes[i].fd_mode,
+ test_modes[i].mode);
else
- ret = get_handles_fopen (filename, fd, fp, test_modes[i].mode);
+ fileret = get_handles_fopen (filename, fd, fp, test_modes[i].mode);
- if (ret != 0)
- return ret;
+ if (fileret != 0)
+ return fileret;
long off = ftell (fp);
if (off != test_modes[i].old_off)
@@ -143,7 +146,12 @@ do_ftell_test (const char *filename)
/* The effect of this write on the offset should be seen in the ftell
call that follows it. */
- int ret = write (fd, data, data_len);
+ int write_ret = write (fd, data, data_len);
+ if (write_ret != data_len)
+ {
+ printf ("write failed (%m)\n");
+ ret |= 1;
+ }
off = ftell (fp);
if (off != test_modes[i].new_off)
@@ -184,21 +192,23 @@ do_write_test (const char *filename)
{
for (int i = 0; i < sizeof (test_modes) / sizeof (struct test); i++)
{
+ int fileret;
printf ("\twrite: %s (file, \"%s\"): ", j == 0 ? "fopen" : "fdopen",
test_modes[i].mode);
if (j == 0)
- ret = get_handles_fopen (filename, fd, fp, test_modes[i].mode);
+ fileret = get_handles_fopen (filename, fd, fp, test_modes[i].mode);
else
- ret = get_handles_fdopen (filename, fd, fp, test_modes[i].fd_mode,
- test_modes[i].mode);
+ fileret = get_handles_fdopen (filename, fd, fp,
+ test_modes[i].fd_mode,
+ test_modes[i].mode);
- if (ret != 0)
- return ret;
+ if (fileret != 0)
+ return fileret;
/* Move offset to just before the end of the file. */
- off_t ret = lseek (fd, file_len - 1, SEEK_SET);
- if (ret == -1)
+ off_t seek_ret = lseek (fd, file_len - 1, SEEK_SET);
+ if (seek_ret == -1)
{
printf ("lseek failed: %m\n");
ret |= 1;
@@ -258,17 +268,20 @@ do_append_test (const char *filename)
{
for (int i = 0; i < sizeof (test_modes) / sizeof (struct test); i++)
{
+ int fileret;
+
printf ("\tappend: %s (file, \"%s\"): ", j == 0 ? "fopen" : "fdopen",
test_modes[i].mode);
if (j == 0)
- ret = get_handles_fopen (filename, fd, fp, test_modes[i].mode);
+ fileret = get_handles_fopen (filename, fd, fp, test_modes[i].mode);
else
- ret = get_handles_fdopen (filename, fd, fp, test_modes[i].fd_mode,
- test_modes[i].mode);
+ fileret = get_handles_fdopen (filename, fd, fp,
+ test_modes[i].fd_mode,
+ test_modes[i].mode);
- if (ret != 0)
- return ret;
+ if (fileret != 0)
+ return fileret;
/* Write some data. */
size_t written = fputs_func (data, fp);