While coding my latest patch I found an annoying const warning when
freeing strings we allocate with talloc but then assign to a variable
of type "const char *".
This made me investigate our talloc_zfree macro and I found a weird
patch, made back in the times that really doesn't make much sense.
I reverted that and then applied a fix to avoid const warnings, as they
are bogus in this case.
I also removed a duplicated macro that was out of sync already.
Simo Sorce (3):
Revert "Avoid accessing half-deallocated memory when using
talloc_zfree macro."
Avoid duplicating macros
Avoid const warnings when deallocating memory
src/ldb_modules/memberof.c | 4 ----
src/util/util.h | 6 +-----
2 files changed, 1 insertions(+), 9 deletions(-)