'va_start' used in function with fixed args

Question

以下のソースコードで、'va_start' used in function with fixed args
と出てしまい、どうしたらいいのか分かりません。
書き方が古いそうなのですが、どう書き直せばよろしいでしょうか？
va_start(args, va_alist);
void lprintf(va_alist)
の引数を指定すればいいと思ったのですが、できませんでした。
C言語に詳しい方、よろしくお願いします。

// エラー内容
daemon.c:89:2: error: 'va_start' used in function with fixed args
        va_start(args, va_alist);
        ^
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/clang/12.0.5/include/stdarg.h:17:29: note: expanded from macro 'va_start'
#define va_start(ap, param) __builtin_va_start(ap, param)
                            ^
1 error generated.

// daemon.c
#include	<stdio.h>
#include	<stdlib.h>
#include	<string.h>
#include	<errno.h>
#include	<syslog.h>
//#include	<varargs.h>
#include    <stdarg.h>
#include	<unistd.h>
#include	<signal.h>
#include	<sys/types.h>
#include	<sys/stat.h>
#include	<fcntl.h>
#include	"daemon.h"

#define	MAXFD	64

static int	IsDaemon=0;

void InitDaemon(char *ident,int chdir_root,int all_fd_close)
{
int	i,fd;
pid_t	pid;

	if((pid=fork())!=0){
		exit(0);
    }
    
	setsid();
	signal(SIGHUP,SIG_IGN);
	if((pid=fork())!=0){
		exit(0);
	}
	
	IsDaemon=1;

	if(chdir_root){
		
		chdir("/");
	}

	if(all_fd_close){
		
		for(i=0;i<MAXFD;i++){
			close(i);
		}
		
		if((fd=open("/dev/null",O_RDWR,0))!=-1) {
			(void)dup2(fd,0);
			(void)dup2(fd,1);
			(void)dup2(fd,2);
			if(fd>2){
				(void)close(fd);
			}
		}
	}

	
	openlog(ident,LOG_PID,LOG_NOTICE);
}


char *str_cut(char *str,int len)
{
static char	*buf=NULL;
static int	buf_len=0;

	if(buf_len<=len){
		if(buf!=NULL){
			free(buf);
		}
		buf_len=len+1;
		buf=(char *)calloc(buf_len,sizeof(char));
	}

	strncpy(buf,str,len);
	buf[len]='\0';

	return(buf);
}


void lprintf(va_alist)
//va_dcl
{
va_list	args;
char	*fmt;
char	buf[513];

	va_start(args, va_alist);
	fmt=va_arg(args,char *);
	vsnprintf(buf,sizeof(buf),fmt,args);
	buf[512]='\0';

	if(IsDaemon){
	}
	else{
		fflush(stdout);
		fputs(str_cut(buf,512),stderr);
		fflush(stderr);
	}
}


void lperror(char *str)
{
char	buf[513];

	snprintf(buf,sizeof(buf),"%s : %s\n",str,strerror(errno));
	buf[512]='\0';

	if(IsDaemon){
	}
	else{
		fflush(stdout);
		fputs(str_cut(buf,512),stderr);
		fflush(stderr);
	}
}

#ifdef	TEST
int main()
{
char	buf[2048];
int	i;
FILE	*fp;

	InitDaemon("daemon:test",1,1);

	fprintf(stderr,"stderr\n");

	lprintf("start\n");

	lprintf("cwd=%s\n",getcwd(buf,sizeof(buf)));

	for(i=0;i<2047;i++){
		buf[i]=i%10+'0';
	}
	buf[2047]='\0';
	lprintf(buf);

	if((fp=fopen("/dev/abcdefg","r"))==NULL){
		lperror("fopen");
	}

	lprintf("end\n");
}
#endif

Answer 1

そうなるとvoid lprintf()の引数はどうすればいいでしょうか？

たとえばこんなかんじ:

#include <stdio.h>
#include <stdarg.h>

void lprintf(const char* format,...) {
  va_list args;
  char    buf[513];

  va_start(args, format);
  int size = vsnprintf(buf, sizeof(buf), format, args);
  buf[size]='\0';

  fputs(buf, stderr);
}

int main() {
  lprintf("[%d : %c]\n", 123, 'A');
  return 0;
}

Comments

[nyamagoyaki]

void lprintf(const char *fmt, ...)
{
va_list args;
char buf[513];

va_start(args, fmt);
fmt=va_arg(args,char *);
vsnprintf(buf,sizeof(buf),fmt,args);
buf[512]='\0';

if(IsDaemon){
}
else{
fflush(stdout);
fputs(str_cut(buf,512),stderr);
fflush(stderr);
}
va_end(args);
}

としてみたのですが、今度は
daemon.c:103:6: error: conflicting types for 'lprintf'
void lprintf(char *fmt, ...)
などと出てしまいます。どこが間違えているのでしょうか？？

[episteme]

このlprintfはヘッダ内(daemon.h?)で宣言されていますか? 宣言と定義は一致していますか?

[nyamagoyaki]

daemon.hでは
void lprintf(const char *fmt);
と定義いたしましたが、
./daemon.h:6:6: note: previous declaration is here
void lprintf(const char *fmt);
と出てしまいます。

[episteme]

一致してないじゃん。

[nyamagoyaki]

すみません。時間を置いたらできました。ありがとうございました！！

[episteme]

時間を置いたら? どゆこと? カップヌードルじゃあるまいし。

Answer 2

va_startを使う場合、それを呼ぶ関数に1つは固定の引数が必要です。それをva_startに渡して、可変長引数へのポインタを得ます（man）。

va_alistはva_dclと組み合わせる、引数の型宣言のない関数の記法にしか使えず、va_startと併用するものではありません。

Comments

[nyamagoyaki]

本では上記のコードのように、va_startとva_alistを併用しているのですが、どう書き直せばよろしいでしょうか？

[maisumakun]

本の正誤表は出ていませんか？

[maisumakun]

あるいは、本の可変長引数の書き方は無視して、自分で...とva_startを使った関数を書いてみる、というのも一案です。

[nyamagoyaki]

出ていないです...
書き直すとなると、va_alistやva_dclはなくなると思いますが、そうなるとvoid lprintf()の引数は
どうすればいいでしょうか？