Переполнение буфера в функции

[demoniqus]

Вот код:

#include <stdio.h>
#include <stdlib.h>
#include </usr/include/postgresql/libpq-fe.h>
#include </usr/include/postgresql/pg_config.h>
#include <string.h>


char *getColNumFromTable(char schema[], char table[]);

int main() {
char schema_str[] = "bil";
char* schema = schema_str;
char table_str[] = ".connect";
char* table = table_str;

printf("\n first_arg = %s\n", schema_str);
printf("\n first_arg = %i\n", *schema_str);
printf("\n first_arg = %s\n", schema);
printf("\n * = %s\n", schema);
getColNumFromTable(schema, table);
}

char *getColNumFromTable(char schema[], char table[]) {
strcat(schema , table);
printf("\n\n function getColNumFromTable  %s \n\n", schema);
printf("\n\n%s%s\n\n", schema, table);
}

функция getColNumFromTable получает два строковых параметра и должна их объединить, а затем вывести на печать. Собственно вывод на печать происходит. Но также я получаю сообщение о переполнении буфера с вылетом на ошибке 134, если суммарная длина двух строк превышает 12 видимых символов. Подскажите, в чем моя ошибка?

[counter]

char schema_str[] = "bil";

память была выделена статически и изменить ее нельзя, а вы пытаетесь туда еще и строку загнать...

а если так?

char schema_str[256] = "bil";

[demoniqus]

Сообщение от counter

память была выделена статически и изменить ее нельзя, а вы пытаетесь туда еще и строку загнать...

а если так?

char schema_str[256] = "bil";

вариант прокатывает, но не хотелось бы делать привязку к какому-то числу, ведь где-нибудь можно и вылезти за его пределы случайно...
а с другой стороны не хотелось бы впустую выделять байты, если они не будут задействованы...
Подскажите, в таком варианте нет подвохов:

char schema_str[(sizeof("billing") + sizeof(".connections") + 2)] = "billing";

И, если я верно понимаю, то при конкатенации строк нулевой символ в конце первой строки не удаляется?

[counter]

Сообщение от demoniqus

вариант прокатывает

это рабочий вариант!

Сообщение от demoniqus

но не хотелось бы делать привязку к какому-то числу, ведь где-нибудь можно и вылезти за его пределы случайно...

и

Сообщение от demoniqus

char schema_str[(sizeof("billing") + sizeof(".connections") + 2)] = "billing";

вроде как одно и тоже

Сообщение от demoniqus

а с другой стороны не хотелось бы впустую выделять байты, если они не будут задействованы...

У вас PDP-11 ? Что вам так жалко пару байт? К тому же, если вы знаете размер данных, с которыми будете работать, то в чем загвоздка?

#define MAX_BUFFER_LENGTH  512
...

char str[MAX_BUFFER_LENGTH];

[demoniqus]

У вас PDP-11 ? Что вам так жалко пару байт? К тому же, если вы знаете размер данных, с которыми будете работать, то в чем загвоздка?

Я заранее не знаю, какой длины будут строки - это может быть и 3 символа и 333...
Свой вариант немножко перепишу

#include <stdio.h>
#include <stdlib.h>
#include </usr/include/postgresql/libpq-fe.h>
#include </usr/include/postgresql/pg_config.h>
#include <string.h>

char *getColNumFromTable(char schema[], char table[], char col_name[]);

int main() {
char sch_str[] = "billing";// значение будет динамическим
char table_str[] = ".connection";// значение будет динамическим
char col_name_str[] = ".connection_column";// значение будет динамическим
char schema_str[(sizeof(sch_str) + sizeof(table_str) + sizeof(col_name_str))] = "";
strcat(schema_str, sch_str);

char* schema = schema_str;
char* col_name = col_name_str;
char* table = table_str;

printf("\n first_arg = %s\n", schema_str);
printf("\n first_arg = %i\n", *schema_str);
printf("\n sizeof = %lu\n", sizeof(schema_str));
printf("\n first_arg = %s\n", schema);
printf("\n * = %s\n", schema);
getColNumFromTable(schema, table, col_name);
}

char *getColNumFromTable(char schema[], char table[], char col_name[]) {
strcat(schema , table);
strcat(schema , col_name);
printf("\n\n function getColNumFromTable  %s \n\n", schema);
printf("\n\n function getColNumFromTable sizeof %lu \n\n", sizeof(schema));
printf("\n\n%s%s\n\n", schema, table);
}

Однако я не понимаю, как тут все работает... По идее, чтобы слепить три переменные, нужно выделить под них место, равное сумме их sizeof'ов, т.е.
char schema_str[(sizeof(sch_str) + sizeof(table_str) + sizeof(col_name_str))] = "";
однако и вариант
char schema_str[(sizeof(sch_str) + 5)] = "";
работает, хотя количество символов в трех переменных явно больше...
И почему в функции getColNumFromTable печатается sizeof = 8?

#define MAX_BUFFER_LENGTH  512
...

char str[MAX_BUFFER_LENGTH];

Мне еще далеко до определения костант...))))

[counter]

Сообщение от demoniqus

Я заранее не знаю, какой длины будут строки - это может быть и 3 символа и 333...

Динамическое выделение памяти используйте.

Сообщение от demoniqus

По идее, чтобы слепить три переменные, нужно выделить под них место, равное сумме их sizeof'ов

Не совсем. Этот sizeof не мешало бы еще и на количество элементов умножить. В Си это работает, потому что sizeof(char) равен 1 байт, поэтому и sizeof(указатель на строку) вернет длину строки.

[demoniqus]

Сообщение от counter

Динамическое выделение памяти используйте.

Вот я это и пытаюсь сделать, да только не понимаю, сколько ж нужно ее выделить...
работает даже так:

char *getColNumFromTable(char table[], char col_name[], char schema[]) {
char result[41] = "";

printf("\n sizeof table %lu %s \n", sizeof(table), table);
printf("\n sizeof col_name %lu %s \n", sizeof(col_name), col_name);
printf("\n sizeof schema %lu %s\n", sizeof(schema), schema);
strcat(result , schema);
strcat(result , table);
strcat(result , col_name);
printf("\n\n RESULT function getColNumFromTable sizeof %lu  %s \n\n", sizeof(result), result);
}

хоть я и задал сразу для result размер 41, но тем не менее, в него без проблем добавились три строки общей длиной 46...

Не совсем. Этот sizeof не мешало бы еще и на количество элементов умножить. В Си это работает, потому что sizeof(char) равен 1 байт, поэтому и sizeof(указатель на строку) вернет длину строки.

у меня sizeof вроде нормально отрабатывает и для кириллицы, считая ее двухбайтовой (т.е. sizeof("ааа") = 7)...

[counter]

Сообщение от demoniqus

Вот я это и пытаюсь сделать, да только не понимаю, сколько ж нужно ее выделить...

Что-то не видно этих попыток.

Сообщение от demoniqus

работает даже так:
...
хоть я и задал сразу для result размер 1, но тем не менее, в него без проблем добавились три строки общей длиной 46...
...
И еще где-то тут ошибка сегментации затесалась...

Да тут что угодно будет. Бред это! Вы так записываете в память, которая может быть занята в любой момент другим процессом (или даже вашим) и тогда все рухнет!

[demoniqus]

Сообщение от counter

Что-то не видно этих попыток.

Я просто не стал выкладывать все нежизнеспособные попытки, а так я испробовал в частности все варианты result от 0 до 46...

Да тут что угодно будет. Бред это! Вы так записываете в память, которая может быть занята в любой момент другим процессом (или даже вашим) и тогда все рухнет!

Вы успели ответить прежде, чем я успел подправить свои слова... вместо 1 получилось 41 и тогда ошибка сегментации исчезла. Однако я пока не понимаю, почему 41, а не общая длина в 46... если только последние 5 байт, как вы говорите, могут переопределиться в любой момент любым другим процессом, если их принудительно не выделить result[46] (или точнее result[(sizeof(...) + sizeof(...) + ...)])

[counter]

Сообщение от demoniqus

Я просто не стал выкладывать все нежизнеспособные попытки, а так я испробовал в частности все варианты result от 0 до 46...

в этом что-то есть

Динамическое выделение памяти: calloc, malloc/free; new/delete - читаем разбираемся.