Как исправить функцию с переменным числом аргументов?

Question

[Алексей Ярков]

Пытался сделать обертку над printf:

void echo(char * const msg, ...)
{
	va_list arguments;
	char *format = (char*)malloc((sizeof(PACKAGE_NAME) + sizeof(PACKAGE_VERSION) + sizeof(msg)) * sizeof(arguments));
	sprintf(format, "[%s-%s]: %s\n", PACKAGE_NAME, PACKAGE_VERSION, msg);
	va_start(arguments, msg);
	vprintf(format, arguments);
	va_end(arguments);
}

// use:
echo("TEST: %d, %d, %d, %d, %d", 1, 2, 3, 4, 5); // вывод: TEST: 1

Я запутался ))) Как исправить функцию?

#UPD1
Почитал ответы и теперь функция выглядит так:

void echo(char * const msg, ...)
{
	va_list arguments;
	va_start(arguments, msg);

	static const char* FORMAT = "[%s-%s]: ";
	char *msgbuf = (char*)malloc(strlen(msg) * sizeof(arguments));
	char *format = (char*)malloc(strlen(FORMAT) + strlen(PACKAGE_NAME) + strlen(PACKAGE_VERSION) + 1);

	vsprintf(msgbuf, msg, arguments);
	sprintf(format, FORMAT, PACKAGE_NAME, PACKAGE_VERSION);
	printf("%s%s\n", format, msgbuf);

	va_end(arguments);
	free(msgbuf);
	free(format);
}

Но магия в другом. Вызывая из разных мест можно получить разные результаты ))) Приведу код без лишнего:

int main (int argc, char * const argv[], char * const argp[])
{
	// Здесь все как надо работает
	Print.echo(_("MSG: %d, %d, %d, %d"), 1, 2, 3, 4);
	Print.echo(_("One of the arguments %s, %s, %s, %s, %s is mandatory"), ArgKeys[4].manstr, ArgKeys[5].manstr, ArgKeys[6].manstr, ArgKeys[7].manstr, ArgKeys[8].manstr);
/**
 * Проверка аргументов и т. п.
 */
	if (strlen(source) > 0)
	{
		while(count >= 1) {
			password = PassGen.getPassword(source, length);
			printf("%s\n", password);
			count--;
		}
	} else {
		// А тут выводится только первый аргумент, а "ArgKeys[5].manstr, ArgKeys[6].manstr, ArgKeys[7].manstr, ArgKeys[8].manstr" игнорируются
		Print.echo(_("One of the arguments %s, %s, %s, %s, %s is mandatory"), ArgKeys[4].manstr, ArgKeys[5].manstr, ArgKeys[6].manstr, ArgKeys[7].manstr, ArgKeys[8].manstr);
		exit(EXIT_FAILURE);
	}

WTF? Как такое возможно?

Comments

[Алексей Ярков]

Ivan Sokolov: Константы ))) Имя программы и версия:

#define PACKAGE_NAME "binname"
#define PACKAGE_VERSION "1.0.0"

[megorit]

Это точно pure C? =)
password = PassGen.getPassword(source, length);

[Алексей Ярков]

Егор Марчук: да =) Это я отсутствие классов компенсирую )))
https://github.com/yarkovaleksei/passgen/blob/mast...

[Алексей Ярков]

Егор Марчук: https://github.com/yarkovaleksei/passgen/blob/mast...

Answer 1

[Mercury13]

char *format = (char*)malloc((sizeof(PACKAGE_NAME) + sizeof(PACKAGE_VERSION) + sizeof(msg)) * sizeof(arguments));

У вас выделяется (и к тому же не освобождается) буфер совершенно рандомного размера. У меня работает и так, но это вопрос везения. Даже если написатьchar format[200]; будет лучше.

1. Более точно (запас допустим, нехватка  — нет) высчитать длину буфера.
2. Освободить его, наконец!

static const char* FORMAT1 = "[%s-%s]: %s\n";
  char *format = (char*)malloc(strlen(FORMAT1) + strlen(msg) + strlen(PACKAGE_NAME) + strlen(PACKAGE_VERSION) + 1);
  sprintf(format, "[%s-%s]: %s\n", PACKAGE_NAME, PACKAGE_VERSION, msg);
  va_start(arguments, msg);
  vprintf(format, arguments);
  va_end(arguments);
  free(format);

Более удачное, но менее универсальное решение — printf заголовок, vprintf строку.

printf("[%s-%s]: ", PACKAGE_NAME, PACKAGE_VERSION);

  va_list arguments;
  va_start(arguments, msg);
  vprintf(msg, arguments);
  va_end(arguments);

  putchar('\n');

UPD1.
Ваша новая ошибка — в коде

char *msgbuf = (char*)malloc(strlen(msg) * sizeof(arguments));

Раз уж хотите так, решение — вычислить длину буфера через snprintf.

Comments

[Алексей Ярков]

В конце вызвать free(format);?

[Mercury13]

Я подкорректировал ответ.

[Mercury13]

Простите, +1 не там написал.

[Алексей Ярков]

Mercury13: я исправил, когда программа упала )

[res2001]

Так же можно вычислять размер достаточного буфера, вызвав sprintf с первым параметром NULL, тогда она не будет никаких действий производить, но вернет вычисленный размер буфера. Потом выделяете этот объем памяти и заполняете его как обычно.
Но в целом, проще так как показал Mercury13

[Mercury13]

res2001: Не нашёл такого поведения sprintf. Чьё-то расширение?

[Mercury13]

res2001: P.S. Такое поведение есть у snprintf.

[Алексей Ярков]

Сделал почти так как вы предложили ))) Но проблема не исчезла. Сейчас обновлю вопрос.

[Mercury13]

Не почти, а выделяются два буфера, и один правильно, второй — нет.

[Mercury13]

sizeof(arguments)) — это принципиально неверно. Это всего лишь размер структуры, призванной развёртывать стек.

[Алексей Ярков]

Mercury13: я понял ошибку. А так правильно?

int buflen = snprintf(NULL, 0, msg, arguments);
	char *msgbuf = (char*)malloc(buflen+1);

[Алексей Ярков]

Алексей Ярков: Отвечу сам: нет, не правильно. Если так написать, то программа выдаст строку и в конце это:

*** Error in `./bin/binname': free(): invalid next size (fast): 0x0000000001088b10 ***
Аварийный останов

[Mercury13]

vsnprintf

Answer 2

[res2001]

По моему с самими аргументами у вас все нормально.
У меня есть похожий код (не мой), успешно работает.
По моему при выделении памяти ошибка, а именно вот этот момент: sizeof(msg)) * sizeof(arguments)
Думаю вместо этого нужно: strlen(msg)*sizeof(char)

Comments

[Алексей Ярков]

Нет, не в этом дело ((

[res2001]

Алексей Ярков: по крайней мере выражение sizeof(msg)) * sizeof(arguments) в вашем контексте - смысла особого не имеет, имхо.