이런 경우는 어떻게 처리하는게 좋나요?

주제를 뭘로할지 마땅치가 않네요.. 현재 간단한 파일 전송 프로그램을 짜고 있습니다.

/*
 * 전송된 데이터 기록
 */
void
data_write(FILE *fp, char *buf, int cnt, int sockfd)
{
	if(write(fp, buf, cnt) != cnt) {
		err_print("data write error");
		finish(fp, sockfd);
	}
}

위 함수는 네트웍으로 전송된 데이터를 넘겨주면 그걸 파일에 기록해주는 역할을 하는데요.. 일단 함수에 넘겨준 내용만큼 쓰기를 못하면 에러를 출력하고 종료하도록 만들었습니다.

finish함수가 열려있는 fd와 소켓을 닫고 종료하는 함수인데요.. 이 함수때문에 data_write함수에 sockfd라는 파라메터가 추가가 되었습니다. 데이터 기록하는 루틴과는 전혀 상관이 없고 단지 에러 발생시 finish함수한테 넘겨주기 위해서 받는건데요..

이런 경우엔 차라리 아래와 같이 함수를 바꿔서..

/*
 * 전송된 데이터 기록
 */
char
data_write(FILE *fp, char *buf, int cnt)
{
	if(write(fp, buf, cnt) != cnt) 
		return -1;
	
	return 0;
}

data_write를 호출하는 쪽에서 에러처리를 하는 방법도 생각을 해봤습니다. 근데 이렇게 하자니 단 몇줄이라도 호출하는 쪽에 더 추가가되고... 차라리 파라메터 하나 더 늘리는게 더 간단한걸까 싶기도하고..

두개중 어떤 방법이 좋을까요?