>Prelude, show us what that code should be changed too
It seems I answer a question of this nature every day. The correct way to do this is to allocate enough memory to hold what you intend to copy to p.
-PreludeCode:char *p = malloc ( sizeof ( "Hello, world!" ) ); if ( p != NULL ) strcpy ( p, "Hello, world!" );