Sscanf warning string buffer overflow как исправить

1. 
   13.06.2016, 15:49
   
   
   #1
   

   

   Пользователь

   

   ---

   sscanf warning: String buffer overflow.
   

   Всем привет. У меня такой вопрос. Как многие знают, я занимаюсь модом Funny RP. Так вот, как написал Daniel_Cortez:

   Сам собой напрашивается вопрос: зачем вообще к моду подключен sscanf2? Ради пары отрывков кода, которые лень переделать?
   Кстати, sscanf2 используется в моде всего 5 раз.
   Вот самое первое место, где он встречается:
   PHP код:
   public OnPlayerCommandText(playerid, cmdtext[])
   {
   new string[256];
   new tmp[128],idx;
   new cmd[128];
   new giveplayerid;
   cmd = strtok(cmdtext,idx);
   // sscanf
   new command[32], params[128];
   sscanf(cmdtext, «s[32]s[128]», command, params);
   Параметры команды отделяются от названия сразу двумя способами, в лучших быдлокодерских традициях.
   И присмотритесь внимательно к форматной строке в sscanf: «s[32]s[128]». В переменную params ничего не запишется, вместо этого и название, и параметры окажутся в command. Мало того, если длина команды и параметров в сумме больше 31 символа, sscanf выплюнет сообщение в лог «sscanf warning: String buffer overflow» — и после этого гадай, откуда берётся такое сообщение.
   В остальных 4 случаях с помощью sscanf2 в командах (OnPlayerCommandText) обрабатываются параметры из params, но поскольку в params ничего нет, то команды будут работать не так, как задумано.

   Ну вот попробовал наконец добавить команду с использованием sscanf и у меня в консоли выбило:

   PHP код:

   
sscanf warning: String buffer overflow. 




   Ну как я понял, переполнение буфера. Но Daniel_Cortez сказал, что sscanf подключен к моду не правильно, по этому будут такие ошибки.

   Так как же правильно подключить sscanf к моду? Я гуглил, везде одно и тоже и в моде тоже самое. Прошу по этому вашей помощи. Заранее благодарю.

   Вот собственно подключение к sscanf.

   PHP код:

   
public OnPlayerCommandText(playerid, cmdtext[])
{
     new string[256];
    new tmp[128],idx;
    new cmd[128];
    new giveplayerid;
    cmd = strtok(cmdtext,idx);
    // sscanf
    new command[32], params[128];
    sscanf(cmdtext, "s[32]s[128]", command, params); 




   Ну а вот тестовая команда:

   PHP код:

   
if(strcmp(cmd, "/msg", true) == 0)
    {
        static const fmt_str[] = "Администратор %s: %s";
        new str[sizeof fmt_str + 16 + MAX_PLAYER_NAME + 50];
        if(PlayerInfo[playerid][pAdmin] < 4) return true;
        if(sscanf(params, "s[50]", params[0])) return SendClientMessage(playerid, COLOR_GREY, "Используйте: /msg [текст]");
        format(str, sizeof(str), fmt_str, PlayerInfo[playerid][pName], params[0]);
        SendClientMessageToAll(0xFFCC00AA, str);
        return true;
    } 




   Если тут написать текст больше 50, то в консоли выбивает про буфер. Я пробовал изменять на 128 и т.д., но все равно, если больше напишешь символов, то будет снова это сообщение. P.S. используется sscanf 2.8.1

   ---

2. 
   15.06.2016, 17:53
   
   
   #2
   

   

   Пользователь

   

   ---

   Ну и? Кто нибудь поможет? Где все профессионалы?

   ---

3. 
   15.06.2016, 18:11
   
   
   #3
   

   

   Пользователь

   

   ---

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

   Ну и? Кто нибудь поможет? Где все профессионалы?

   Поэтому и надо делать ограничение на кол-во символов, понятное же дело

   ---

4. 
   15.06.2016, 21:59
   
   
   #4
   

   

   Пользователь

   

   ---

   Код:

   if(sscanf(params, "?<SSCANF_QUIET=1>s[50]", params[0])) return SendClientMessage(playerid, COLOR_GREY, "Используйте: /msg [текст]");

   ---


Пользователь сказал cпасибо:


5. 
   16.06.2016, 09:18
   
   
   #5
   

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

   Так как же правильно подключить sscanf к моду?

   PHP код:

   
#include <sscanf2> 




   Худшее, что здесь можно сделать — неправильно написать название инклуда или забыть скопировать сам инклуд в pawno/include. В любом случае компилятор укажет на такую ошибку.

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

   Вот собственно подключение к sscanf.

   PHP код:

   
public OnPlayerCommandText(playerid, cmdtext[])
{
     new string[256];
    new tmp[128],idx;
    new cmd[128];
    new giveplayerid;
    cmd = strtok(cmdtext,idx);
    // sscanf
    new command[32], params[128];
    sscanf(cmdtext, "s[32]s[128]", command, params); 




   «Подключение» и «использование» — совершенно разные вещи…

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

   Ну а вот тестовая команда:

   PHP код:

   
if(strcmp(cmd, "/msg", true) == 0)
    {
        static const fmt_str[] = "Администратор %s: %s";
        new str[sizeof fmt_str + 16 + MAX_PLAYER_NAME + 50];
        if(PlayerInfo[playerid][pAdmin] < 4) return true;
        if(sscanf(params, "s[50]", params[0])) return SendClientMessage(playerid, COLOR_GREY, "Используйте: /msg [текст]");
        format(str, sizeof(str), fmt_str, PlayerInfo[playerid][pName], params[0]);
        SendClientMessageToAll(0xFFCC00AA, str);
        return true;
    } 




   Если тут написать текст больше 50, то в консоли выбивает про буфер.

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

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

   Я пробовал изменять на 128 и т.д., но все равно, если больше напишешь символов, то будет снова это сообщение.

   Каждая строка должна заканчиваться нуль-символом » (должна же машина как-то знать, где заканчивается последовательность символов), под который нужна ещё одна ячейка в массиве. В итоге в OPCT массивы command и params должны быть оба размером в 129 ячеек.

   Есть ещё вариант посложнее, но использующий только один массив в 129 ячеек вместо двух. Вместо того, чтобы использовать sscanf2, можно скопировать всё содержимое cmdtext в массив command, пройтись циклом по массиву, найти первый пробел и записать на его месте », затем пропустить лишние пробелы между командой и параметрами (да, их там может быть больше одного) и записать позицию, где начинаются параметры команды, в переменную params_start_pos. В итоге сама команда будет в массиве command, а вместо params можно будет использовать command[params_start_pos].
   А вообще для подобных манипуляций есть командные процессоры, кои я и советую вам использовать.

   Индивидуально в ЛС по скриптингу не помогаю. Задавайте все свои вопросы

   здесь (click)

   .

   ---


Пользователь сказал cпасибо:


6. 
   16.06.2016, 18:59
   
   
   #6
   

   

   Пользователь

   

   ---

   new cmd[64],params[64],idx;
   cmd = strtok(cmdtext, idx);
   sscanf(cmdtext,»s[32]s[64]»,cmd,params);

   ---


Пользователь сказал cпасибо:


7. 
   17.06.2016, 13:07
   
   
   #7
   

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

   new cmd[64],params[64],idx;
   cmd = strtok(cmdtext, idx);
   sscanf(cmdtext,»s[32]s[64]»,cmd,params);

   Команда может быть длиннее 31 символа, а параметры — длиннее 63 (сразу будет заметно в чатовых командах типа /s, /a, /pm, /sms и т.п.).

   Индивидуально в ЛС по скриптингу не помогаю. Задавайте все свои вопросы

   здесь (click)

   .

   ---

8. 
   17.06.2016, 13:24
   
   
   #8
   

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

   Команда может быть длиннее 31 символа, а параметры — длиннее 63 (сразу будет заметно в чатовых командах типа /s, /a, /pm, /sms и т.п.).

   Почему 63? Разве длинна строки параметра — это не остаток от лимита ввода в чат (128) минус лимит длины имени функции (31)?

   ---

9. 
   17.06.2016, 13:37
   
   
   #9
   

   Сообщение от Anton Styazhkin
   

   Почему 63? Разве длинна строки параметра — это не остаток от лимита ввода в чат (128) минус лимит длины имени функции (31)?

   Именно.
   Кстати, по поводу моего первого поста в этой теме, небольшая поправка: если команда может быть длиной от 2 до 128 символов, то максимальная длина параметров будет равна 129 — 2 — 1 = 126 символов. В итоге под массив params потребуется не 129, а 126 ячеек.

   Индивидуально в ЛС по скриптингу не помогаю. Задавайте все свои вопросы

   здесь (click)

   .

   ---

10. 
    17.06.2016, 13:44
    
    
    #10
    

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

    Именно.
    Кстати, по поводу моего первого поста в этой теме, небольшая поправка: если команда может быть длиной от 2 до 128 символов, то максимальная длина параметров будет равна 129 — 2 — 1 = 126 символов. В итоге под массив params потребуется не 129, а 126 ячеек.

    Учет пробелов и ‘/’ всегда должен быть, если ты об этом.

    ---

Пользователи, просматривающие эту тему

Ваши права

#1

 

Отправлено 23 апреля 2017 — 22:39

0

#2

Отправлено 24 апреля 2017 — 00:23

sscanf warning: String buffer overflow

0

#3

Отправлено 24 апреля 2017 — 13:04

sscanf warning: String buffer overflow

0

#4

Отправлено 24 апреля 2017 — 14:02

0

#5

Отправлено 24 апреля 2017 — 14:53

0

#6

Отправлено 24 апреля 2017 — 15:48

0

#7

Отправлено 24 апреля 2017 — 19:11

0

#8

Отправлено 24 апреля 2017 — 19:43

0

#9

Отправлено 24 апреля 2017 — 20:52

0

#10

Отправлено 24 апреля 2017 — 21:26

-1

Вопросы

stock LoadOther()
{
	new time = GetTickCount();
	format(query,sizeof(query), "SELECT * FROM `"TABLE_OTHER"`");
	mysql_query(query);
	mysql_store_result();
	if(mysql_fetch_row(query) == 1)
	{
		mysql_fetch_row(query);
		sscanf(query, "p<|>iiiiiiiiiiiiis[40]iiiiiiiiiiiiiiiiiiiiiiiiiii",
		FracBank[0],
		FracBank[1],
		FracBank[3],
		FracBank[4],
		FracBank[5],
		FracBank[6],
		FracBank[7],
		FracBank[8],
		FracBank[9],
		sklads,
		ugol,
		drev,
		ed,
		NewsPrice,
		Mats[1],//Police
		Mats[2],//DPC
		Mats[0],///Army
		Mats[4],//FSB
		Mats[5],//GOP
		Mats[6],//LB
		Mats[8],//BB
		Mats[9],//UGBRATVA
		Mineral[0],//mineral
		Metal,
		Mineral[1],
//		FracBank[11],
		Mats[16],
		Mats[12],
		Mats[18],
		MafiaMineral[0],
		MafiaMineral[1],
		MafiaMineral[2],
		DrugsBank[F_GOPOTA],
		DrugsBank[F_LYTBRATVA],
		DrugsBank[F_BATBRATVA],
		DrugsBank[F_UGBRATVA],
		DrugsBank[F_YAKUZA],
		DrugsBank[F_LCN],
		DrugsBank[F_RM],
		FuelBank,
		Plavnya,
		TOTALPRIZ);
		sscanf(NewsPrice, "p<,>iiiiii",SmsPrice[0], SmsPrice[1], CallPrice[0], CallPrice[1], AdvertPrice[0], AdvertPrice[1]);
	}
	mysql_free_result();
	printf("ARPLOAD: Успешная загрузка 'LoadOther' за %d м.с",GetTickCount() - time);
	return true;
}

stock LoadOther()
{
	new time = GetTickCount();
	format(query,sizeof(query), "SELECT * FROM `"TABLE_OTHER"`");
	mysql_query(query);
	mysql_store_result();
	if(mysql_fetch_row(query) == 1)
	{
		mysql_fetch_row(query);
		sscanf(query, "p<|>iiiiiiiiiiiiis[40]iiiiiiiiiiiiiiiiiiiiiiiiiiii",
		FracBank[0],
		FracBank[1],
		FracBank[3],
		FracBank[4],
		FracBank[5],
		FracBank[6],
		FracBank[7],
		FracBank[8],
		FracBank[9],
		sklads,
		ugol,
		drev,
		ed,
		NewsPrice,
		Mats[1],//Police
		Mats[2],//DPC
		Mats[0],///Army
		Mats[4],//FSB
		Mats[5],//GOP
		Mats[6],//LB
		Mats[8],//BB
		Mats[9],//UGBRATVA
		Mineral[0],//mineral
		Metal,
		Mineral[1],
//		FracBank[11],
		Mats[16],
		Mats[12],
		Mats[18],
		MafiaMineral[0],
		MafiaMineral[1],
		MafiaMineral[2],
		DrugsBank[F_GOPOTA],
		DrugsBank[F_LYTBRATVA],
		DrugsBank[F_BATBRATVA],
		DrugsBank[F_UGBRATVA],
		DrugsBank[F_YAKUZA],
		DrugsBank[F_LCN],
		DrugsBank[F_RM],
		FuelBank,
		Plavnya,
		TOTALPRIZ);
		sscanf(NewsPrice, "p<,>iiiiii",SmsPrice[0], SmsPrice[1], CallPrice[0], CallPrice[1], AdvertPrice[0], AdvertPrice[1]);
	}
	mysql_free_result();
	printf("ARPLOAD: Успешная загрузка 'LoadOther' за %d м.с",GetTickCount() - time);
	return true;
}

stock LoadOther()
{
	new time = GetTickCount();
	format(query,sizeof(query), "SELECT * FROM `"TABLE_OTHER"`");
	mysql_query(query);
	mysql_store_result();
	if(mysql_fetch_row(query) == 1)
	{
		mysql_fetch_row(query);
		sscanf(query, "p<|>iiiiiiiiiiiiis[128]iiiiiiiiiiiiiiiiiiiiiiiiiii", //41
		FracBank[0],
		FracBank[1],
		FracBank[3],
		FracBank[4],
		FracBank[5],
		FracBank[6],
		FracBank[7],
		FracBank[8],
		FracBank[9],
		sklads,
		ugol,
		drev,
		ed,
		NewsPrice,
		Mats[1],//Police
		Mats[2],//DPC
		Mats[0],///Army
		Mats[4],//FSB
		Mats[5],//GOP
		Mats[6],//LB
		Mats[8],//BB
		Mats[9],//UGBRATVA
		Mineral[0],//mineral
		Metal,
		Mineral[1],
//		FracBank[11],
		Mats[16],
		Mats[12],
		Mats[18],
		MafiaMineral[0],
		MafiaMineral[1],
		MafiaMineral[2],
		DrugsBank[F_GOPOTA],
		DrugsBank[F_LYTBRATVA],
		DrugsBank[F_BATBRATVA],
		DrugsBank[F_UGBRATVA],
		DrugsBank[F_YAKUZA],
		DrugsBank[F_LCN],
		DrugsBank[F_RM],
		FuelBank,
		Plavnya,
		TOTALPRIZ);
		sscanf(NewsPrice, "p<,>iiiiii",SmsPrice[0], SmsPrice[1], CallPrice[0], CallPrice[1], AdvertPrice[0], AdvertPrice[1]);
	}
	mysql_free_result();
	printf("ARPLOAD: Успешная загрузка 'LoadOther' за %d м.с",GetTickCount() - time);
	return true;
}

"p<|>iiiiiiiiiiiiis[128]iiiiiiiiiiiiiiiiiiiiiiiiiii"

Let’s suppose I have this:

CMD:whisper(playerid, params[])
{
    new whisp[90];
    if(sscanf(params, "us[90]", giveplayerid, whisp)) return SendClientMessage //Blahblahblah whatever
    //Do something here
    return 1;
}

Well, it’s a command. So, anybody can use it. Now, if the string text overpass the string size, the sscanf plugin prints this warning: sscanf warning: string buffer overflow.

Now, is there a way to avoid it BEFORE parsing the string with sscanf? Or is it normal to do this warning? I can see that sscanf cuts the string at the point, but are the warnings bad? Or can I just live with it?

I am pretty sure this was fixed in sscanf2, How old is the version of sscanf your using? mine was from 03rd of November 2010, I had downloaded it but forgot to

#include <sscanf>
 to 
#include <sscanf2>

up until two days ago. Imagine my face when half my problems instantly disappeared.

#include <sscanf>
 to 
#include <sscanf2>

Yeah, I use sscanf2.

Increase the string size in sscanf for the «whisp» variable, apparently 90 cells isn’t enough!

Duh, I know that, but as it’s a command, there’s always that MORON who will type dpfamdfsgposdifmbspofvibmxpocvibmxpoifgmspodifmgps odifmgpsodifmg just to fill in the entire space in the chatbox. This will overflow the array. So what? Would the only solution be increasing ALL of my strings’ sizes to 128? I’m definitely not doing that and fucking my stack.

Have you ever coded something in a server before? Do you know how often I type messages over 128 characters? I’ll answer that for you- VERY often! The maximum size of any strings sent to the client is 128 cells, you may as well allow people to use them all. Honestly, the main problem is creating a 5000 cell string and really only needing 100 cells of it.

See this for more information: https://sampforum.blast.hk/showthread.php?tid=55261

I’ma talking about smaller strings, like an identifier, like «type enter to enter». A string to hold ENTER would need only 6 cells, so why would I make 128 cells for a string that needs only 6? Just to get around the warning?

You would just add a check underneath the sscanf line…?

if(strlen(params) > 6)
     return 0;

Here’s what I did originally (Yea yea I used godfather, moving on..)

new x_nr[16];
    if(sscanf(params, "s[16] ", x_nr))
    {
        SendClientMessage(playerid, COLOR_YELLOW2, "USO: /irc [Nome]");
        SendClientMessage(playerid, COLOR_YELLOW2, "Nomes disponнveis: Entrar, MOTD, Senha, Precisasenha, Trancar, Kick, Status");
        return 1;
    }
    if(strlen(x_nr)) > 16) return SendClientMessage("blahblahblah");

Posts: 6,129
Threads: 36
Joined: Jan 2009

It’s a warning to you as a scripter to increase the size of your string, as it truncates the text. It’s nothing you should consider serious really, as it manages the buffer overflow itself, it just warns you that it had to.