Thread: function definition with structure node

  1. #1
    Registered User
    Join Date
    Sep 2001

    function definition with structure node

    I need to write a definition of a function with all the following points to be addressed:

    1. function name is Problem
    2. it receives a parameter named b of the struct node * type
    3. it creates a new dynamic variable with "p" pointing to it and with contents the same as those of the node to which "b" points to
    4. it returns the address of the newly created code

    not sure if I have this right, please critique as necessary:

    void Problem(struct node * b)
         struct node * p = (struct node *) malloc (sizeof (struct node));
         p = b;
         returns p;

  2. #2
    Banned master5001's Avatar
    Join Date
    Aug 2001
    Visalia, CA, USA
    This pretty much accomplishes what you wanted, however, this could have memory leaks. Lets say your program works like this:
    void Problem(struct node * b){
         struct node *p = (struct node *) malloc (sizeof (struct node));
         p = b;
         returns p;
    int main(void){
         struct node *mem_leak = (struct node *)malloc(sizeof(struct node));
         return 0;
    Okay, now if you are not very experienced the memory leak probably went under your radar but I'll explain. You allocated a block of memory to struct node *mem_leak via malloc. Immediately afterward you pointed it to a different block of data. The other block of data that it was pointing to in the first place didn't just disappear. So what you may want to do is free a struct before passing it in toe Problem.

  3. #3
    Also, you want your prototype to allow you to return a pointer.


    typedef struct
    int data1;
    long data2;

    nodeP Problem(nodeP);
    int main(void);

    int main(void)
    nodeP rootNode;
    nodeP newNode;

    rootNode = 0L; /* init to known state */
    newNode = 0L;

    rootNode = malloc(sizeof(node)); /* allocate */
    newNode = Problem(rootNode); /* duplicate root node */
    newNode = 0L;

    rootNode = 0L;


    nodeP Problem(nodeP b);
    nodeP theNodeP;

    theNodeP = 0L; /* init to known state */
    theNodeP = malloc(sizeof(node)); /* allocate it */
    theNodeP->data1 = b->data1; /* copy struct contents (brute force) */
    theNodeP->data2 = b->data2;



    anyway, something like the above.


Popular pages Recent additions subscribe to a feed

Similar Threads

  1. Screwy Linker Error - VC2005
    By Tonto in forum C++ Programming
    Replies: 5
    Last Post: 06-19-2007, 02:39 PM
  2. Replies: 28
    Last Post: 07-16-2006, 11:35 PM
  3. Problem with Visual C++ Object-Oriented Programming Book.
    By GameGenie in forum C++ Programming
    Replies: 9
    Last Post: 08-29-2005, 11:21 PM
  4. Request for comments
    By Prelude in forum A Brief History of
    Replies: 15
    Last Post: 01-02-2004, 10:33 AM
  5. Interface Question
    By smog890 in forum C Programming
    Replies: 11
    Last Post: 06-03-2002, 05:06 PM