smb_iconv modifies constant input strings

[Andrew Bartlett]

On Sat, 2003-09-27 at 11:20, Jeremy Allison wrote:
> On Sat, Sep 27, 2003 at 11:17:16AM +1000, Andrew Bartlett wrote:
> > 
> > Neither of these options are very good.  Samba's use of iconv() really
> > does assume it behaves in the way descriped in the linux manpage - which
> > declares it as char ** input, but does not modify the input string.  
> 
> I'm not so sure. Changing smb_iconv to take a char **, not const char **
> seems the easiest. If iconv does behave this way then we need to honour
> that interface.

Sure - and if it really did behave that way (it doesn't according to
Matt's other post), then we would be in much more trouble.  What does
your patch achieve except moving we get the warnings?  I think we would
need to strdup the inputs if we really wanted to honer this interface...

Andrew Bartlett

-- 
Andrew Bartlett                                 abartlet at pcug.org.au
Manager, Authentication Subsystems, Samba Team  abartlet at samba.org
Student Network Administrator, Hawker College   abartlet at hawkerc.net
http://samba.org     http://build.samba.org     http://hawkerc.net
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.samba.org/archive/samba-technical/attachments/20030927/9d0fba1d/attachment.bin