cvs.gedasymbols.org/archives/browse.cgi | search |
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!
webmaster | delorie software privacy |
Copyright © 2019 by DJ Delorie | Updated Jul 2019 |