Bug 562393 – g_buffered_input_stream_read_byte broken if data available

2008-11-28  Matthias Clasen  <mclasen@redhat.com>

        Bug 562393 – g_buffered_input_stream_read_byte broken if data
        available

        * gio/gbufferedinputstream.c (g_buffered_input_stream_read_byte): Fix
        handling of buffered content.  Patch by Philip Withnall

        * gio/tests/buffered-input-stream.c: Add a testcase for this bug.
        * gio/tests/Makefile.am: And build it



svn path=/trunk/; revision=7686
This commit is contained in:
Matthias Clasen 2008-11-28 05:57:07 +00:00 committed by Matthias Clasen
parent 636fa4640d
commit 0b715510de
4 changed files with 77 additions and 2 deletions

View File

@ -1,3 +1,14 @@
2008-11-28 Matthias Clasen <mclasen@redhat.com>
Bug 562393 g_buffered_input_stream_read_byte broken if data
available
* gio/gbufferedinputstream.c (g_buffered_input_stream_read_byte): Fix
handling of buffered content. Patch by Philip Withnall
* gio/tests/buffered-input-stream.c: Add a testcase for this bug.
* gio/tests/Makefile.am: And build it
2008-11-28 Matthias Clasen <mclasen@redhat.com>
Bug 561807 inotify_sub.c :: dup_dirname() fails to remove trailing

View File

@ -878,7 +878,7 @@ g_buffered_input_stream_read_byte (GBufferedInputStream *stream,
available = priv->end - priv->pos;
if (available < 1)
if (available != 0)
{
g_input_stream_clear_pending (input_stream);
return priv->buffer[priv->pos++];

View File

@ -23,7 +23,8 @@ TEST_PROGS += \
g-file-info \
data-input-stream \
data-output-stream \
g-icon
g-icon \
buffered-input-stream
if OS_UNIX
TEST_PROGS += live-g-file unix-streams desktop-app-info
@ -50,6 +51,9 @@ data_output_stream_LDADD = $(progs_ldadd)
g_icon_SOURCES = g-icon.c
g_icon_LDADD = $(progs_ldadd)
buffered_input_stream_SOURCES = buffered-input-stream.c
buffered_input_stream_LDADD = $(progs_ldadd)
live_g_file_SOURCES = live-g-file.c
live_g_file_LDADD = $(progs_ldadd)

View File

@ -0,0 +1,60 @@
/* GLib testing framework examples and tests
* Copyright (C) 2008 Red Hat, Inc.
* Authors: Matthias Clasen <mclasen@redhat.com>
*
* This work is provided "as is"; redistribution and modification
* in whole or in part, in any medium, physical or electronic is
* permitted without restriction.
*
* This work 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.
*
* In no event shall the authors or contributors be liable for any
* direct, indirect, incidental, special, exemplary, or consequential
* damages (including, but not limited to, procurement of substitute
* goods or services; loss of use, data, or profits; or business
* interruption) however caused and on any theory of liability, whether
* in contract, strict liability, or tort (including negligence or
* otherwise) arising in any way out of the use of this software, even
* if advised of the possibility of such damage.
*/
#include <glib/glib.h>
#include <gio/gio.h>
#include <stdlib.h>
#include <string.h>
static void
test_read_byte (void)
{
GInputStream *base;
GInputStream *in;
g_test_bug ("562393");
base = g_memory_input_stream_new_from_data ("abcdefghijk", -1, NULL);
in = g_buffered_input_stream_new (base);
g_assert_cmpint (g_buffered_input_stream_read_byte (G_BUFFERED_INPUT_STREAM (in), NULL, NULL), ==, 'a');
g_assert_cmpint (g_buffered_input_stream_read_byte (G_BUFFERED_INPUT_STREAM (in), NULL, NULL), ==, 'b');
g_assert_cmpint (g_buffered_input_stream_read_byte (G_BUFFERED_INPUT_STREAM (in), NULL, NULL), ==, 'c');
g_assert_cmpint (g_input_stream_skip (in, 3, NULL, NULL), ==, 3);
g_assert_cmpint (g_buffered_input_stream_read_byte (G_BUFFERED_INPUT_STREAM (in), NULL, NULL), ==, 'g');
}
int
main (int argc,
char *argv[])
{
g_type_init ();
g_test_init (&argc, &argv, NULL);
g_test_bug_base ("http://bugzilla.gnome.org/");
g_test_add_func ("/buffered-input-stream/read-byte", test_read_byte);
return g_test_run();
}