summaryrefslogtreecommitdiff
path: root/source3/Makefile.in
diff options
context:
space:
mode:
authorAndrew Tridgell <tridge@samba.org>2004-02-02 12:02:43 +0000
committerAndrew Tridgell <tridge@samba.org>2004-02-02 12:02:43 +0000
commit423ad9097448e5a07ba26172302450ae7165d85a (patch)
tree7c6cc18230c045346176483f038cd708a1ef7f5c /source3/Makefile.in
parent590b60045d184a84659cfcb13c6a1a5039b51954 (diff)
downloadsamba-423ad9097448e5a07ba26172302450ae7165d85a.tar.gz
samba-423ad9097448e5a07ba26172302450ae7165d85a.tar.bz2
samba-423ad9097448e5a07ba26172302450ae7165d85a.zip
the conversion from int to size_t in charcnv did not take into account
one place where we checked "if (src_len > 0)". I actually would greatly prefer to switch back to int for src_len. The type *can* be negative, which means an unsigned type is inappropriate. There is absolutely no reason why "int" should not be used for a parameter like this. I didn't change back to int as we are close to a release and I wanted a mininal change, but please don't go changing types like this in future without very careful testing and a damn good reason. this bug broke pull_ucs2(), I would not be surprised if it caused all sorts of nastiness. Thanks to vl for noticing the symptoms! (This used to be commit 8b8f0c527959c707f274b435fc59156e37232981)
Diffstat (limited to 'source3/Makefile.in')
0 files changed, 0 insertions, 0 deletions