NetNews Usenet Archive 1992 #16/NN_1992_16.iso/spool/comp/unix/programm/3936

Text File | 1992-07-27 | 1.4 KB | 41 lines

Newsgroups: comp.unix.programmer
Path: sparky!uunet!walter!att!cbfsb!cbnewsf.cb.att.com!rajeev
From: [e-mail] (rajeev.dolas)
Subject: Re: I am stumped on this one!!!
Message-ID: <[e-mail]>
Sender: [e-mail]
Organization: AT&T
References: <[e-mail]> <[e-mail]> <[e-mail]>
Distribution: usa
Date: Mon, 27 Jul 1992 13:53:50 GMT
Lines: 28

In article <[e-mail]> [e-mail] (Guy Harris) writes:
>>>        cp_line = (char *)malloc(sizeof(strlen(pars_wbuf)));
>>
>> You're allocing the sizeof an int, which is likely four bytes.
>> You also forgot to alloc space for the '\0'. Try it again with:
>>
>>    cp_line = (char*)malloc(sizeof(strlen(pars_wbuf) + 1));
>
>No, try it again with:
>
>>    cp_line = (char*)malloc(strlen(pars_wbuf) + 1);
>
>because, as you noted, the "sizeof" doesn't belong there - it turns the
>argument into the size of the result of "strlen", which isn't the length
>of the string.
>
>(Also, try not leaving every single line of the original posting in your
>followup; it makes it rather hard to find the stuff that *doesn't* come
>from the original posting....)

I did modify my code and removed the sizeof(), but that does not help.
As I mentioned previously, the error is in fputs() or putc().

Thanks.

Raj Dolas