Re: CVS update: subversion/subversion/libsvn_fs txn-table.c

jimb@tigris.org wrote:

> User: jimb
> Date: 01/01/24 14:13:28
>
> Modified: subversion/libsvn_fs txn-table.c
> Log:
> * libsvn_fs/txn-table.c (svn_fs__get_txn_list): Don't set *NAMES_P
> until we know the operation has succeeded.

[snip]

> - (*names_p)[names_count++] = apr_pstrndup (pool, key.data, key.size);
> + names[names_count++] = key.data;

You didn't run fs-test before checking this in, by any chance? key.data
isn't null-terminated, so you get garbage at the end of the transaction
ID. That's why I copied out the key.

If you really think it's worth the trouble to save space/cycles here, we
could do some magic about how we set up the DBT's buffer. For myself I
think it's just not worth the trouble right now.

-- 
Brane �ibej
    home:   <brane_at_xbc.nu>             http://www.xbc.nu/brane/
    work:   <branko.cibej_at_hermes.si>   http://www.hermes-softlab.com/
     ACM:   <brane_at_acm.org>            http://www.acm.org/