Re: svn commit: r11155 - in trunk/subversion: clients/cmdline include libsvn_client libsvn_ra_dav libsvn_ra_local libsvn_ra_svn libsvn_repos mod_dav_svn svnserve tests/clients/cmdline/getopt_tests_data

Philip Martin <philip@codematters.co.uk> writes:

> Of course values greater than INT_MAX won't occur if you are starting
> with an int in the first place... which raises a question about
> ra_svn_log2: it's probably not valid to pass an int to
> svn_ra_svn_write_tuple with an "n" format, as that will attempt to
> retrieve 8 bytes from the varargs when only 4 bytes have been passed.

$ valgrind svn log svn://localhost/repo

==31356== Conditional jump or move depends on uninitialised value(s)
==31356== at 0x404544F7: conv_10_quad (apr_snprintf.c:397)
==31356== by 0x404555BC: apr_vformatter (apr_snprintf.c:846)
==31356== by 0x40465FEB: apr_pvsprintf (apr_pools.c:1604)
==31356== by 0x402FFFAD: writebuf_printf (marshal.c:214)
==31356==
==31356== Conditional jump or move depends on uninitialised value(s)
==31356== at 0x404544F9: conv_10_quad (apr_snprintf.c:397)
==31356== by 0x404555BC: apr_vformatter (apr_snprintf.c:846)
==31356== by 0x40465FEB: apr_pvsprintf (apr_pools.c:1604)
==31356== by 0x402FFFAD: writebuf_printf (marshal.c:214)
==31356==
==31356== Conditional jump or move depends on uninitialised value(s)
==31356== at 0x40467FF5: __udivdi3 (libgcc2.c:802)
==31356== by 0x40454580: conv_10_quad (apr_snprintf.c:431)
==31356== by 0x404555BC: apr_vformatter (apr_snprintf.c:846)
==31356== by 0x40465FEB: apr_pvsprintf (apr_pools.c:1604)
==31356==
==31356== Conditional jump or move depends on uninitialised value(s)
==31356== at 0x404545B1: conv_10_quad (apr_snprintf.c:435)
==31356== by 0x404555BC: apr_vformatter (apr_snprintf.c:846)
==31356== by 0x40465FEB: apr_pvsprintf (apr_pools.c:1604)
==31356== by 0x402FFFAD: writebuf_printf (marshal.c:214)
==31356==
==31356== Conditional jump or move depends on uninitialised value(s)
==31356== at 0x40021163: strlen (mac_replace_strmem.c:164)
==31356== by 0x402FFFBB: writebuf_printf (marshal.c:216)
==31356== by 0x4030060A: svn_ra_svn_write_number (marshal.c:361)
==31356== by 0x4030087D: vwrite_tuple (marshal.c:417)
==31356==
==31356== Conditional jump or move depends on uninitialised value(s)
==31356== at 0x4002116A: strlen (mac_replace_strmem.c:164)
==31356== by 0x402FFFBB: writebuf_printf (marshal.c:216)
==31356== by 0x4030060A: svn_ra_svn_write_number (marshal.c:361)
==31356== by 0x4030087D: vwrite_tuple (marshal.c:417)
==31356==
==31356== Syscall param write(buf) contains uninitialised or unaddressable byte(s)
==31356== at 0x4077A404: __libc_write (in /usr/lib/debug/libc-2.2.5.so)
==31356== by 0x4045CBF9: apr_socket_send (sendrecv.c:40)
==31356== by 0x402FFD1D: writebuf_output (marshal.c:156)
==31356== by 0x402FFE84: writebuf_flush (marshal.c:184)
==31356== Address 0x41E79104 is 4160 bytes inside a block of size 8244 alloc'd
==31356== at 0x40028B88: malloc (vg_replace_malloc.c:153)
==31356== by 0x40465BD6: pool_alloc (apr_pools.c:1277)
==31356== by 0x402FF8E3: svn_ra_svn_create_conn (marshal.c:50)
==31356== by 0x402F82BB: ra_svn_open (client.c:586)

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org