Revert "tdbtool: fix off-by-one error in argument length. (bug #2344)"
authorMichael Adam <obnox@samba.org>
Wed, 9 Jul 2008 10:48:05 +0000 (12:48 +0200)
committerMichael Adam <obnox@samba.org>
Wed, 9 Jul 2008 10:59:37 +0000 (12:59 +0200)
commitc09b42746411138d9cb16eb69fe78c3a2826924e
tree8b48b230011eef44ac164dd0582a01503f1d3d5f
parentf587e0310c802109f2647fc8637db4b01c6a49d3
Revert "tdbtool: fix off-by-one error in argument length. (bug #2344)"

This reverts commit fafb8ad2b81b9a46cf8259bedc1dca5023b06115.

This fix is not valid:

1. convert_string() is not only used for key strings but also for data.

2. Some databases use string_tdb_data() i.e. non-null-terminated strings
   as keynames and others (like the one I was using), use
   string_term_tdb_data(), i.e. zero-terminated key strings.

After discussion with Metze, the easiest (and proper way) to
handle this is to specify key names as "keyname\0" for databases
which use string_term_tdb_data().

Sorry for the noise...

Michael
(cherry picked from commit 17c012c4645f4e9542537c15f80d9b4e74304d11)
(This used to be commit 42c953c106b3da90e8c3452c4a246392502c200d)
source3/lib/tdb/tools/tdbtool.c