getline

[kir_rik]

Здравствуйте.
Натолкнулся на проблемы с cin.getline. При введении большего кол-ва символов, чем дозволяет второй параметр, команда в цикле на второй раз выполняться уже категорически отказывается, в результате цикл загоняется. Кто виноват и что делать?

char pass[7] = "qwerty";
char buf[7];    
while (1){
          printf("enter password: ");
          cin.getline(buf, 7);    
          if(!strcmp(buf,pass)) { 
              printf("true\n");  
              break;
          }  
          else  
          printf("false\n"); 
}

[|{ot]

команда в цикле на второй раз выполняться уже категорически отказывается

Какая команда?

После

cin.getline(buf, 7);

поставь

cin.ignore(...);

[rik_nel]

char buf[256];
...
cin.getline(buf, 256);

Решит проблему.

[|{ot]

Решит проблему.

А если больше 255 символов кто то введет??
Потом ошибки в программе, переполнение буфера.. эксплоиты и т.д.??

[rik_nel]

Ну да,вы правы,что то я поспешыл с выводом)

[kir_rik]

2|{ot
getline второй раз видимо не выполняется. Во всяком случае если ввести более 7 символов, то цикл в дальнейшем будет шпарить false-ами ничего от пользователя не запрашивая.
А что писать-то в параметры ignore? cin.ignore(7,' '); не помогло. Как и cin.ignore(7,'\n'); По-моему это немного не та степь.

2rik_nel Здорово конечно, но мне нужно исправить уязвимость (изначально вместо getline стоял gets), не меняя самих переменных. К тому же такое решение вызовет проблему на strcmp

О, ну вот. вроде как разрулил.
После гетлайна надо было добавить

if (cin.fail()) 
    cin.clear();

getline при переполнении выставляет бит состояния =1. Его проверяем fail'ом. А потом подчищаем буфер и приходит счастье

Спасибо за участие

[rik_nel]

А если так ?

#include <string>
#include <conio.h>
#include <iostream>
int main()
{
char pass[7] = "qwerty";
char buf[7];    
do
{
          std::cout << "enter password: ";
          std::cin >> buf; 
               }while((strcmp(pass,buf) != 0));
               std::cout << "true" << std::endl;
getch(); 
}

[kir_rik]

2rik_nel
Красиво, но....
я ввожу "1234567890123456aaa", затем "aaa" получаю true

[rik_nel]

ГГм,и че он так ?

[kir_rik]

Это стек. Мы объявили пасс раньше чем буф. Значит у буфа адреса младше. Размер буфа больше 2, но меньше 16. Под него отведено 16 байт, а сразу за ним лежит пасс. Вводим 16 символов чтобыдобраться до пасс и еще что-нибудь чтобы его затереть
Как я уже написал, проблема решается через cin.fail(). Ну или что более находчиво - объявлением буф раньше, чем пасс