Stack around the variable 's' was corrupted

@fantaz1 · Регистрация: 08.11.2008

Author24 — интернет-сервис помощи студентам

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

Run-Time Check Failure #2 - Stack around the variable 's' was corrupted.

Пример кода:

C++

#include <stdio.h>
 
int main()
{   
    char s[6];  
    scanf("%s",s);
    return 0;
}

@grizlik78 · 05.07.2011, 17:56

Вводится 6 или более символов?

@co6ak · 05.07.2011, 18:04

ога. за пределы вылез
после мятого символа - enter (шестой )

@fantaz1 · 05.07.2011, 18:10 **[ТС]**

мда тупонул конкретно.. про ентер забыл) бывает..

@grizlik78 · 05.07.2011, 18:16

Безопасный способ (правда лишние символы остаются в буфере)

C

1	scanf("%5s",s);

@fasked · 05.07.2011, 18:56

grizlik78, вот так не будут оставаться:

C

1	scanf("%5s%*s", str);

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

@grizlik78 · 05.07.2011, 19:09

fasked, да, но счастья-то всё-равно нет

при вводе: 123 456 789
в строку правильно считается 123, а вот пропустится 456, хотя хотелось просто ограничить первое слово. В общем всё может оказаться сложнее.

@easybudda · 05.07.2011, 19:11

C

1	scanf("%5[^\n]%*c", s)

@grizlik78 · 05.07.2011, 19:19

А так и вовсе через пробел нельзя вводить. Да и если строка длинее 5, то только один символ выбрасывается из лишних, а остальные опять в следующую попадают. Вообще, пользователю, конечно, по башке надо надавать за некорректный ввод и программу завершить

@fasked · 05.07.2011, 19:27

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

Вообще, пользователю, конечно, по башке надо надавать за некорректный ввод и программу завершить

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

@grizlik78 · 05.07.2011, 19:34

Ну вот неплохо было бы, если при помощи одного только scanf можно было реализовать нечто, аналогичное следующему коду:

C

char buf[7];
scanf("%6s", buf);
if (strlen(buf) > 5)
  printf("Ты - негодяй, Израэль Хэнкс!\n");

@easybudda · 05.07.2011, 22:04

grizlik78,

C

1	if ( scanf("%6s", buf) == 1 && buf[5] )

@grizlik78 · 05.07.2011, 22:11

easybudda, ну, положим, проверять возврат scanf надо и в моём коде. Я тоже немножко негодяй

Но buf[5] не эквивалент strlen(buf) > 5, так как прочитанное слово может оказаться коротким. Это перед scanf надо 0 в buf[5] писать. Да, тогда немножко эффективнее (что здесь не главное), но по-прежнему громоздко. Интересно, где смысл понятнее будет, в strlen(buf) или в buf[5]?
Но сам scanf, насколько понимаю, не позволяет определить перебор с заданной длинной строки.

@easybudda · 05.07.2011, 22:29

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

Но buf[5] не эквивалент strlen(buf) > 5, так как прочитанное слово может оказаться коротким.

Если сначала строку не обнулить - то да, фигня получится.

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

Но сам scanf, насколько понимаю, не позволяет определить перебор с заданной длинной строки.

не-а, просто вернёт 1, если в строку что-то запишется...

accept · 06.07.2011, 06:23

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

C

1	if ( scanf("%6s", buf) == 1 && buf[5] )

ab cd

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

Ну вот неплохо было бы, если при помощи одного только scanf можно было реализовать нечто, аналогичное следующему коду:

а идея кода в чём ?

C

1 2	if (scanf("%5s%с", buf, &end) == 2 && end == '\n') ;

@grizlik78 · 06.07.2011, 09:24

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

а идея кода в чём ?

В том, что при вводе допустима любая строка (точнее слово) длиной от 0 до 5 символов. Ввод более длинного слова — ошибка.

@easybudda · 06.07.2011, 09:24

accept,

C

#include <stdio.h>
    
int main(void){
    char buf[7] = { 0 };
    
    printf("> ");
    if ( scanf("%6s", buf) == 1 && buf[5] )
        printf("You bastard, Israel Henks!\n");
    else
        printf("Just %s\n", buf);
    
    return 0;
}

Код

$ ./israel
> ab cd
Just ab

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

а идея кода в чём ?

Да чёрт его знает...

accept · 07.07.2011, 02:21

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

В том, что при вводе допустима любая строка (точнее слово) длиной от 0 до 5 символов. Ввод более длинного слова — ошибка.

а, ну значит там неправильно написано
%6s - это ввод шести символов, а нуль-символ сохраняется всегда

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

C

1	char buf[7] = { 0 };

а зачем 7 ?

можно ещё так

C

1	char buf[6] = "";

@grizlik78 · 07.07.2011, 02:26

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

а, ну значит там неправильно написано
%6s - это ввод шести символов, а нуль-символ сохраняется всегда

Правильно-правильно. Требуется ограничить ввод пятью символами и распознать ситуацию, когда вводится больше. Поэтому стоит %6s. А в буфере 7 байт как-раз потому, что '\0' сохраняется всегда.

@easybudda · 07.07.2011, 02:27

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

а зачем 7 ?

опять же

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

Да чёрт его знает...

получилось так, не принципиально, можно и 6 написать...

@grizlik78 2381 / 1665 / 279 Регистрация: 29.05.2011 Сообщений: 3,399
	05.07.2011, 22:11	13
	easybudda, ну, положим, проверять возврат scanf надо и в моём коде. Я тоже немножко негодяй Но buf[5] не эквивалент strlen(buf) > 5, так как прочитанное слово может оказаться коротким. Это перед scanf надо 0 в buf[5] писать. Да, тогда немножко эффективнее (что здесь не главное), но по-прежнему громоздко. Интересно, где смысл понятнее будет, в strlen(buf) или в buf[5]? Но сам scanf, насколько понимаю, не позволяет определить перебор с заданной длинной строки. 0

@easybudda Модератор 12454 / 7479 / 1752 Регистрация: 25.07.2009 Сообщений: 13,755
	05.07.2011, 22:29	14
	Сообщение от grizlik78 Но buf[5] не эквивалент strlen(buf) > 5, так как прочитанное слово может оказаться коротким. Если сначала строку не обнулить - то да, фигня получится. Сообщение от grizlik78 Но сам scanf, насколько понимаю, не позволяет определить перебор с заданной длинной строки. не-а, просто вернёт 1, если в строку что-то запишется... 0

@grizlik78 2381 / 1665 / 279 Регистрация: 29.05.2011 Сообщений: 3,399
	05.07.2011, 17:56	2
	Вводится 6 или более символов? 0

@co6ak Кошковед 521 / 509 / 63 Регистрация: 12.04.2010 Сообщений: 1,390
	05.07.2011, 18:04	3
	ога. за пределы вылез после мятого символа - enter (шестой ) 1

@fantaz1 33 / 25 / 7 Регистрация: 08.11.2008 Сообщений: 107
	05.07.2011, 18:10 [ТС]	4
	мда тупонул конкретно.. про ентер забыл) бывает.. 0

@grizlik78 2381 / 1665 / 279 Регистрация: 29.05.2011 Сообщений: 3,399
	05.07.2011, 19:09	7
	fasked, да, но счастья-то всё-равно нет при вводе: 123 456 789 в строку правильно считается 123, а вот пропустится 456, хотя хотелось просто ограничить первое слово. В общем всё может оказаться сложнее. 0

@grizlik78 2381 / 1665 / 279 Регистрация: 29.05.2011 Сообщений: 3,399
	05.07.2011, 19:19	9
	А так и вовсе через пробел нельзя вводить. Да и если строка длинее 5, то только один символ выбрасывается из лишних, а остальные опять в следующую попадают. Вообще, пользователю, конечно, по башке надо надавать за некорректный ввод и программу завершить 0

@fasked 5043 / 2622 / 241 Регистрация: 07.10.2009 Сообщений: 4,310 Записей в блоге: 1
	05.07.2011, 19:27	10
	Сообщение от grizlik78 Вообще, пользователю, конечно, по башке надо надавать за некорректный ввод и программу завершить Однако, тоже не самое лучшее решение. Хотя... главное, чтобы программа совсем не падала, а хоть как-то сообщала об ошибке. 0

@grizlik78 2381 / 1665 / 279 Регистрация: 29.05.2011 Сообщений: 3,399
	06.07.2011, 09:24	16
	Сообщение от accept а идея кода в чём ? В том, что при вводе допустима любая строка (точнее слово) длиной от 0 до 5 символов. Ввод более длинного слова — ошибка. 0

@grizlik78 2381 / 1665 / 279 Регистрация: 29.05.2011 Сообщений: 3,399
	07.07.2011, 02:26	19
	Сообщение от accept а, ну значит там неправильно написано %6s - это ввод шести символов, а нуль-символ сохраняется всегда Правильно-правильно. Требуется ограничить ввод пятью символами и распознать ситуацию, когда вводится больше. Поэтому стоит %6s. А в буфере 7 байт как-раз потому, что '\0' сохраняется всегда. 0

@easybudda Модератор 12454 / 7479 / 1752 Регистрация: 25.07.2009 Сообщений: 13,755
	07.07.2011, 02:27	20
	Сообщение от accept а зачем 7 ? опять же Сообщение от easybudda Да чёрт его знает... получилось так, не принципиально, можно и 6 написать... 0