FreeTDS Development Group

["Winkless, Geoff" <geoff.winkless AT pera.com>]

Brian wrote:
> Not to mention lack of comments. ;-) I could have incremented 
> the pointers
> directly to save a bit, but figured I'd go with the understandable
> version.
> There should be a terminator added before returning too, oops!
> > >    outstr = malloc(strlen(outstr) + need + 1);
> > > > >    for (i=0;i<strlen(instr);i++) {

Yeuck. Really you should do

for (i=0; instr[i]!=0; i++) {
....

in both loops, and 

outstr = malloc(i + need);

instead of using strlen()

Geoff