Quote Originally Posted by Salem View Post

You need to make sure your new_node->next is NULL.

> SinglyLinkedListNode *new_node = (SinglyLinkedListNode*)malloc(sizeof(SinglyLinkedL istNode));
If this really is C++, then you should be using new, not malloc.
Thanks. fixed those but there still is the segfault. It looks like my lack of knowledge about the rest of the existing code might be what is getting in my way.