Re: realloc problem, corrupt last item

Igal said:

> hay, i have this werid problem with my book adding function, this how
> it looks
> 
> book* AddBook(book *bp, unsigned *size) {
> ...
> //then i use realloc to allocate space for the new item in the bp
> pointer
> bp = (book*)realloc(bp, sizeof(book));
> ...
> }

You have not provided sufficient information for me to help you 
effectively. On general principles, lose the cast, replace sizeof(book) 
with sizeof *bp, use a temp to catch the result of realloc in case it 
fails and returns NULL, and make sure you have #included <stdlib.h>. Note 
that bp is a copy of the calling function's pointer; the original will not

be updated automatically, so if you want to keep the new value you must 
provide a mechanism for the caller to assign this new value to the 
original pointer.

For more specific help, post more specific code. "..." doesn't say
anything 
useful, and you haven't shown the calling code.

<snip>

-- 
Richard Heathfield <http://www.cpax.org.uk>
Email: -http://www.
+rjh@[EMAIL PROTECTED]
 users: <http://www.cpax.org.uk/prg/writings/googly.php>
"Usenet is a strange place" - dmr 29 July 1999