현재 포인터의 위치를 잠시 기억해둘수는 없나요?

포인터형 변수에 저장되는 값은 숫자입니다.
p = p2;
라는 문장은 p 변수에 p2 값을 대입하라는 이상의 의미가 없고,
원래 포인터 값이 저장되는 게 맞습니다.

그보다는 알고리즘을 손보셔야 할 듯합니다.
변수형들에 대해서도 좀 생각해보시고요.

일단 문자열의 마지막에 널문자를 넣는 부분이 잘못되었습니다. 문자 상수가 아니라 문자열을 의도하신 듯 합니다만, 문자상수이기 때문에 잘못된 값이 들어갔을 것입니다. 아마도 경고가 발생했을텐데요.

그리고, strcpy( p++, "1" ) 같은 표현보다는 *p++ = '1'; 정도가 적당하지 않을까합니다. 굳이 복사당하는 문자열의 끝을 알 필요도 없는 상황이니까요.

상황마다 틀리겠지만 p2 는 정상적으로 최초의 포인터 값을 저장하고 있을테지만 출력할 때 널문자를 찾을 때까지 (재수없어서) '\r' 같은게 있었다면 원래 값이 안보일 수도 있습니다.

일단 strcpy( p, '\0' ) 을 "0" 로 바꾸시면 일단 동작은 할 것 같습니다.

오... 답을 하기 위해 대기하고 있는 이 수많은 kldp 요원들~~~~ -_-; 동시에 올라오는 답글 러쉬가 대단하군요...

C에서 '와 "는 전혀 다릅니다. :-) 이건 제가 설명하는 것보다 The C Programming Language 같은 책을 읽으셔서 확실히 알고 가시는 게 더 좋을 것 같습니다.
그리고 "0", '0', '\0' 은 전혀 다른 녀석들입니다. 제일 앞의 녀석은 0다음에 nul이 붙어 있는 곳을 가리키는 포인터이고, 두번째 녀석은 ASCII 코드로 0x30, 세번째 녀석은 ASCII코드 0x00, 즉 nul character입니다.

일은 많지만 (게으르고... 일하기 싫어서) 심심해서 손님께서 만드신 코드의 문제점을 한번 살펴보았습니다. 문제점이라기보다는 저라면 이렇게 할 것 같다는 것이니 오해는 마시고요.

일단 p2 변수가 선언되지 않았으므로 컴파일 에러가 날 것 같습니다.
각 비트가 1인지 0인지 판단하는 부분이 중복되는 이유가 분명하지 않습니다. 만약 그 이유가 strcpy 와 strcat 의 차이라고 해서 코딩했다면 지금 같은 경우는 동일한 동작이므로 차이가 없습니다.
문자를 하나 복사하는데, 문자열 복사하는 함수를 호출하는 것은 낭비입니다.
1을 감소시키는 표현은 --max 로 충분할 것 같습니다.
max 라는 이름이 (매크로 등으로) 문제가 될 소지가 있습니다.
비교당하는 값을 변수로 만들어서 최상위 비트를 세팅한 후에 오른쪽으로 쉬프트하는게 어떨까요?
free(p) -> free(p2) 해야겠지요.
마지막으로 동적 할당이 과연 필요할까요?

	p = malloc(max + 1);

그런데 제가 보기에 굳이 malloc()을 쓸 이유가 없군요. 그냥 배열을 쓰는 편이 쉽습니다.

void 
bin_print(unsigned int u) 
{ 
    int max = sizeof(unsigned int) * CHAR_BIT; 
    char p[max + 1];
    /* ... */
} 

여기 있던 글, 제 실수였기에 삭제했습니다. (실험삼아 돌려보다 다른 데서 에러 난 걸 착각했습니다. :oops:) 읽으신 분들께 죄송합니다.