From:	Clemens Valens <c DOT valensNOc DOT SPAM AT mindless DOT com DOT invalid>
Subject:	Re: URGENT!!! Please (A Little Long)
Newsgroups:	comp.os.msdos.djgpp
Message-ID:	<004aa0e3.faebcda4@usw-ex0101-006.remarq.com>
Lines:	38
Bytes:	1064
X-Originating-Host:	195.154.148.69
Organization:	http://www.remarq.com: The World's Usenet/Discussions Start Here
References:	<3813e85f DOT 18243975 AT news DOT pasteur DOT dialix DOT com DOT au>
X-Wren-Trace:	eJy5kZCJzoTP38aCmNCYm52vgZCXnNCPioPdnJqFxY3Kz5LMx4jHw8TF1s/L
Date:	Mon, 25 Oct 1999 01:35:06 -0700
NNTP-Posting-Host:	10.0.2.6
X-Complaints-To:	wrenabuse AT remarq DOT com
X-Trace:	WReNphoon4 940841021 10.0.2.6 (Mon, 25 Oct 1999 01:43:41 PDT)
NNTP-Posting-Date:	Mon, 25 Oct 1999 01:43:41 PDT
To:	djgpp AT delorie DOT com
DJ-Gateway:	from newsgroup comp.os.msdos.djgpp
Reply-To:	djgpp AT delorie DOT com

> quick fix I have done the following.
> int main(void)
> {
>     STUDENT *Start = NULL;
>
>     Start = AddStudent(Start);
>     CleanUp(Start);
>
>     return 0;
> }
> STUDENT *AddStudent(STUDENT *Student)
> {
>     STUDENT *New;
>     New = malloc(sizeof(STUDENT));
>     /*
>       * etc etc.
>       */
>     Start->Next = New;
>     return (Student);
> }

You mess horribly with pointers. And your quick fix doesn't even
compile since you define Start in main() and you use it in AddStudent()
as if it was a global variable, which it isn't.

You call AddStudent with Student = Start = NULL. Then you create New
which you assign to Start->Next, i.e. NULL->Next (gp fault) and then
you return Student (which is still NULL) and assign it to Start.

I won't give you the final answer, because I think it is your homework,
but you should be able to figure it out now.

Clemens



* Sent from RemarQ http://www.remarq.com The Internet's Discussion Network *
The fastest and easiest way to search and participate in Usenet - Free!

- Raw text -