Linked Lists? Wowser...Any suggestions

Code:

typedef struct dep_db
{
    char name[NAME_LENGTH + 1];
    char ssn[SSN_LENGTH + 1];
    struct dep_db* next;
} DEP_DB_T;

void print_links( const DEP_DB_T* ptr );
DEP_DB_T* get_links( void);


DEP_DB_T* get_links( void )
{
    DEP_DB_T *current, *first;
    int response;

    current = first = malloc( sizeof(DEP_DB_T) );
    printf ("\n\n\tEnter First Item --> ");
    scanf ( "%s", current -> name );

    printf ("\n\n\n\tAdd Another? --> 1==y 0==n");
    scanf ("%d", &response);

    while (response)
    {
        if ((current->next = malloc( sizeof ( DEP_DB_T ) ) ) == NULL )
        {
            printf ("Youre out of memory");
            return first;
        }
        current = current -> next;

        printf ("\n\n\tNAME: ");
        scanf ( "%s", current -> name );

        printf ("\n\n\n\tAdd Another? --> 1==y 0==n");
        scanf ("%d", &response);
    }
    current -> next = NULL;
    return first;
}

void print_links ( const DEP_DB_T* ptr )
{
    int count = 1;
    printf ("\n\n\n" );
    while ( ptr != NULL)
    {
        printf ("\nnumber %d is %s.", count++, ptr -> name);
        ptr = ptr -> next;
    }
}

When I try to compile i get the following

ANSI C++ forbids implicit conversion from "void *" in assignment?

I am so confused...

Please Help!!

- shane

Try compiling as a C program instead of C++.

Quzah.

I figured it out...

Needed to explicitly type the return value

- shane

Originally posted by shaeng
I figured it out...

Needed to explicitly type the return value

- shane

No. You need to do what I said. Stop compiling as C++. This is a common issue posted here. C++ requires you typecast your void pointes. C does not. Compile your program correctly and you'll avoid the problem. It is not recommended you typecast your malloc returns in C.

Quzah.

Quzah,

thanks for the tip.
You were right, sometimes a solution that is found is not the best way to go.

A practice I need to start ingraining in my head.

thanks

-shane

Originally posted by quzah
It is not recommended you typecast your malloc returns in C.

why?

Read this for malloc casting.

Also, do a forum search, this comes up loads of times.