[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [Issue 826] New - Dumper bug (Property-content-length: 0) and fix

From: mark benedetto king <bking_at_inquira.com>
Date: 2002-07-29 18:59:50 CEST

On Mon, Jul 29, 2002 at 02:54:45AM -0000, issues@subversion.tigris.org wrote:
> http://subversion.tigris.org/issues/show_bug.cgi?id=826
> +
> +
> + There were two problems. The obvious bug was that apr_off_t variables
> + were being printed using APR_SIZE_T_FMT. The less obvious bug was that
> + proplen was using apr_off_t when the prop string is stored in memory
> + (unnecessary), and also content_length was only apr_size_t when it needs to
> + hold both textlen and proplen. As textlen is (correctly) apr_off_t,
> + content_length needs to be apr_off_t too.

To prevent this from happening again, we should probably add (I think this
is the correct syntax):

Index: subversion/include/svn_io.h
===================================================================
--- subversion/include/svn_io.h
+++ subversion/include/svn_io.h Sun Jul 28 23:17:19 2002
@@ -350,7 +350,9 @@
 svn_error_t *svn_stream_printf (svn_stream_t *stream,
                                 apr_pool_t *pool,
                                 const char *fmt,
- ...);
+ ...)
+ __attribute__ ((format (printf, 3, 4)));
+
 
 /* Allocate *STRINGBUF in POOL, and read one line from STREAM into it.
    The '\n' is read from the stream, but is not added to the end of

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Jul 29 19:11:34 2002

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.