왜 이렇게 메모리 초기화을 하는지요?

시간이 지나서 이미 해결하신 듯 하지만 ..

위의 리스트 구조체 뒤에 Data Block를 붙여놓은 형태네요.

: list_t : Data Block :

위와 같은 구조를 malloc 으로 할당하고
malloc(sizeof(list_t) + strlen(data.string) + 1))

list_t의 멤버인 .string 이 Data Block의 처음을 가리키려는 것입니다.
(char *)(new_node + sizeof(list_t));

다시 말하자면 new_node 로부터 list_t만큼 떨어진 곳의 주소이므로

Data Block의 처음을 가리키겠다는 의도입니다.

add_data 함수에 두 가지 정도 오류가 있습니다.

받은 data 변수에 원하는 string이 들어 있습니다만, 그것이 새로 할당되는 list에 복사되지는 않는 군요.

새로 할당된 list 변수가 return 되어야할 것 같은데 혹은, global linked list에 들어가야할 것 같은데 pointer를 놓치고 있습니다.

코드를 뜯어보진 않았지만, 메모리 누수가 있다면, 코드 자체의 문제일 것이구요...

이 방법은 오히려 메모리 낭비를 줄이는 방법입니다. 예를 들어, 좀 극단적인 얘기지만, 1바이트를 10번할당한 것과, 10바이트를 한번 할당한 것은 후자가 메모리를 덜 잡아먹으니까요. 지금 예는 두번 할당을 한번할당으로 줄였지요.

어쨌든, 그런 이유로 위와같이 한 것인데, 위의 예는 1바이트 문자열에 대해 그렇게 해서 다행히 문제가 없지만, 2바이트 이상의 타입인 경우는 bus error 가 발생할 수도 있습니다.

대개는 다음과 같은 방법이 더 자주 쓰입니다. 제시하신 예처럼 문자열이 하나라면 포인터 멤버를 둘 필요도 없어서 메모리가 더 절약되죠.

struct list_t {
    ...;
    char s[1];
};

char* s = "Hello, world";
struct list_t* l = (struct list_t*)
    malloc( sizeof(struct list_t)+strlen(s) );
strcpy( l->s, s );