Ошибка ввода float чисел

@squareroot · Регистрация: 19.10.2019

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

Есть вот такой код.

C++ (Qt)

1 2	int result = sscanf(line.c_str(), "v %lf %lf %lf", &v.p.x, &v.p.y, &v.p.z);

Вот такую строку не распознает (под Линукс)
v 439847.000000 422137.000000 79700.000000
А вот с такой проблем нет:
v 439847 422137 79700
В винде все работает. Когда запускаю под отладчиком в линуксе тоже все работает, что удивительно.
Я в инете не нашел, чтоб кто то жаловался. Означает ли это баг либы libstdc++ ?
Если да, то лечить надо регуляркой ?

И заодно хотел узнать, насколько уместны и безопасны все эти сишные функции в плюсовом коде ?

@DrOffset · 01.07.2021, 21:15

squareroot, локаль скорее всего стоит русская. Проверить просто - заменить точку на запятую. Вряд ли баг.

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

насколько уместны и безопасны все эти сишные функции в плюсовом коде ?

Зависит от особенностей вашего проекта.
Абстрактно - ничего плохого в них нет, если использовать правильно.

@squareroot · 01.07.2021, 21:26 **[ТС]**

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

squareroot, локаль скорее всего стоит русская. Проверить просто - заменить точку на запятую. Вряд ли баг.

Вы были правы. Это невероятно даже о таком догадаться.

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

Зависит от особенностей вашего проекта.
Абстрактно - ничего плохого в них нет, если использовать правильно.

Первый минус я нашел - распознавание зависит от текущей локали.
А какие еще сюрпризы мне ждать ?
Там в основе вариативная функция, которая выглядит небезопасной и устаревшей с появлением вариативных шаблонов.
Думаю std::getline + std::regex + конвертеры строка-число выиграют у sscanf по скорости, хотя надо мерить.

@avgoor · 01.07.2021, 21:42

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

Первый минус я нашел - распознавание зависит от текущей локали.

Это не минус. Локаль для этого и предназначена.

@squareroot · 01.07.2021, 22:06 **[ТС]**

Странно, но почему тогда выражение:

C++ (Qt)

1
2
3

    const char point =
            std::use_facet< std::numpunct<char> >
            (std::cout.getloc()).decimal_point();

возвращает точку, а не запятую ?

@DrOffset · 01.07.2021, 22:24

squareroot, это другое. В смысле у cout своя локаль, а scanf использует глобальную.

Добавлено через 2 минуты

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

Думаю std::getline + std::regex + конвертеры строка-число выиграют у sscanf по скорости

Очень вряд ли.

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

Там в основе вариативная функция, которая выглядит небезопасной и устаревшей с появлением вариативных шаблонов.

Переубеждать тут вас никто не будет, не подходит, значит не подходит. Дело ваше же.

@squareroot · 01.07.2021, 22:40 **[ТС]**

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

Переубеждать тут вас никто не будет, не подходит, значит не подходит. Дело ваше же.

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

Добавлено через 12 минут
Если это глобальная локаль:

C++ (Qt)

1
2
3

    const char point =
            std::use_facet< std::numpunct<char> >
            (std::locale()).decimal_point();

то тоже мимо.

@DrOffset · 01.07.2021, 22:45

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

то тоже мимо

Ну вот из-за таких вот высказываний и не хочется совершено ни о чем дискутировать.

Это не глобальная локаль, а временный объект локали.

Глобальная - это вот:
https://en.cppreference.com/w/... ale/global

@squareroot · 01.07.2021, 22:49 **[ТС]**

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

Ну вот из-за таких вот высказываний и не хочется совершено ни о чем дискутировать.

Это не глобальная локаль, а временный объект локали.

Глобальная - это вот:
https://en.cppreference.com/w/... ale/global

Я смотрел эту функцию. Она нужна чтоб менять локаль, А не получать.
Так сказано в документации.
А то что я привел, то там черным по белому написано:

Default constructor. Constructs a copy of the global C++ locale

@DrOffset · 01.07.2021, 22:52

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

А то что я привел, то там черным по белому написано

Вот именно, вы сами-то прочитайте что там написано.

Создает копию глобальной локали. Копию!
Еще раз, не создает глобальную локаль, а создает копию глобальной локали.

@squareroot · 01.07.2021, 22:54 **[ТС]**

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

Вот именно, вы сами-то прочитайте что там написано.

Создает копию глобальной локали. Копию!
Еще раз, не создает глобальную локаль, а создает копию глобальной локали.

Ну Да, копию. Подлинную полную копию создает ведь ? А мне надо только узнать точка или запятая.
Кто по копии мне узнать мешает ?

@DrOffset · 01.07.2021, 22:57

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

А не получать.

Зачем ее получать-то? Чтобы повлиять на поведение функции, нужно наоборот ее задать.

Во-вторых, если вы уже читаете документацию, то там как бы четко написано, что в программе на С++ две локали, С и С++. Это разные вещи. Совпадать они начнут только после вызова функции global.

Добавлено через 1 минуту

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

Кто по копии мне узнать мешает ?

Вы по копии С++ локали хотите узнать настройки локали С?

Я такого сразу предположить не смог, извините.

@squareroot · 01.07.2021, 23:02 **[ТС]**

DrOffset, Ой, ну и заморочек с этой локалью. Ок. Буду искать как вытащить запятую/точку из Си локали.

@DrOffset · 01.07.2021, 23:05

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

Буду искать как вытащить запятую/точку из Си локали.

https://en.cppreference.com/w/cpp/locale/lconv

@squareroot · 01.07.2021, 23:08 **[ТС]**

Готово:

C++ (Qt)

1	const char point = localeconv()->decimal_point;

@squareroot · 02.07.2021, 14:24 **[ТС]**

DrOffset, А может sscanf приводить к сбоям в работе при некорректной строке ?
Надо ли предварительно проверять строку регуляркой ?

Добавлено через 5 минут
Вот даже в инете есть коечто на этот счет:
https://stackoverflow.com/ques... rogramming

Добавлено через 22 минуты
И Википедия тоже не молчит:

scanf is vulnerable to format string attacks. Great care should be taken to ensure that the formatting string includes limitations for string and array sizes. In most cases the input string size from a user is arbitrary and cannot be determined before the scanf function is executed. This means that uses of %s placeholders without length specifiers are inherently insecure and exploitable for buffer overflows. Another potential problem is to allow dynamic formatting strings, for example formatting strings stored in configuration files or other user-controlled files. In this case the allowed input length of string sizes cannot be specified unless the formatting string is checked beforehand and limitations are enforced. Related to this are additional or mismatched formatting placeholders which do not match the actual vararg list. These placeholders might be partially extracted from the stack or contain undesirable or even insecure pointers, depending on the particular implementation of varargs.

Добавлено через 5 часов 24 минуты
Интересная ситуювина.
А sscanf_s под линукс нету ?

Добавлено через 7 минут
макроопределение добавил
#define __STDC_WANT_LIB_EXT1__ 1

@DrOffset · 02.07.2021, 15:31

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

А может sscanf приводить к сбоям в работе при некорректной строке ?

Может.

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

Надо ли предварительно проверять строку регуляркой ?

Скорее надо использовать соответствующие форматные параметры, и(или) проверять размеры входящих строк.

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

А sscanf_s под линукс нету ?

Пока нет.

@squareroot · 02.07.2021, 16:10 **[ТС]**

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

Может.
Скорее надо использовать соответствующие форматные параметры, и(или) проверять размеры входящих строк.

Тут надо разделить ситуации.
1)Сбой из-за неверного типа переменной результата для данной распознаваемой строки и сбой из-за того что пришла некорректная строка для парсинга и эта некорректность безотносительна
того какой тип параметра переменной результата ожидается. Грубо говоря, подали что то совсем нечитабельное(какой бинарный код например) и функция глюкнулась.
Вы дали ответ только по первому случаю.

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

Пока нет.

Я нарыл либу libsafe_iop вроде она по этой теме " ISO TR24731 Bounds Checking Interface"
Щас поковыряю ее и отпишусь.

@DrOffset · 02.07.2021, 16:14

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

. Грубо говоря, подали что то совсем нечитабельное(какой бинарный код например)

Это общая проблема, она не касается конкретно scanf. Scanf обрубит до первого нуля этот бинарный код, если это как строка подавалось и все. Вообще любой пользовательский ввод, если это важно, нужно защищать от такого.

Добавлено через 15 секунд

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

1)

А где 2) ?

@squareroot · 02.07.2021, 16:20 **[ТС]**

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

Это общая проблема, она не касается конкретно scanf. Scanf обрубит до первого нуля этот бинарный код, если это как строка подавалось и все. Вообще любой пользовательский ввод, если это важно, нужно защищать от такого.

Вот я и хотел знать надо ли защищать. В данном случае скорее не пользовательский ввод, а процес чтения файла, где может появится чтото непотребное. И не будет ли лишней это ? Ведь sscanf тоже что то проверяет. А проверка регуляркой недешевая операция.

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

Добавлено через 15 секунд

А где 2) ?

Это оно и есть.

@squareroot 13 / 13 / 1 Регистрация: 19.10.2019 Сообщений: 607
	01.07.2021, 22:54 [ТС]	11
	Сообщение от DrOffset Вот именно, вы сами-то прочитайте что там написано. Создает копию глобальной локали. Копию! Еще раз, не создает глобальную локаль, а создает копию глобальной локали. Ну Да, копию. Подлинную полную копию создает ведь ? А мне надо только узнать точка или запятая. Кто по копии мне узнать мешает ? 0

@DrOffset 18840 / 9839 / 2408 Регистрация: 30.01.2014 Сообщений: 17,280
	02.07.2021, 15:31	17
	Сообщение от squareroot А может sscanf приводить к сбоям в работе при некорректной строке ? Может. Сообщение от squareroot Надо ли предварительно проверять строку регуляркой ? Скорее надо использовать соответствующие форматные параметры, и(или) проверять размеры входящих строк. Сообщение от squareroot А sscanf_s под линукс нету ? Пока нет. 0

@squareroot 13 / 13 / 1 Регистрация: 19.10.2019 Сообщений: 607
	02.07.2021, 16:10 [ТС]	18
	Сообщение от DrOffset Может. Скорее надо использовать соответствующие форматные параметры, и(или) проверять размеры входящих строк. Тут надо разделить ситуации. 1)Сбой из-за неверного типа переменной результата для данной распознаваемой строки и сбой из-за того что пришла некорректная строка для парсинга и эта некорректность безотносительна того какой тип параметра переменной результата ожидается. Грубо говоря, подали что то совсем нечитабельное(какой бинарный код например) и функция глюкнулась. Вы дали ответ только по первому случаю. Сообщение от DrOffset Пока нет. Я нарыл либу libsafe_iop вроде она по этой теме " ISO TR24731 Bounds Checking Interface" Щас поковыряю ее и отпишусь. 0

@DrOffset 18840 / 9839 / 2408 Регистрация: 30.01.2014 Сообщений: 17,280
	01.07.2021, 21:15	2
	Сообщение было отмечено squareroot как решение Решение squareroot, локаль скорее всего стоит русская. Проверить просто - заменить точку на запятую. Вряд ли баг. Сообщение от squareroot насколько уместны и безопасны все эти сишные функции в плюсовом коде ? Зависит от особенностей вашего проекта. Абстрактно - ничего плохого в них нет, если использовать правильно. 1

@squareroot 13 / 13 / 1 Регистрация: 19.10.2019 Сообщений: 607
	01.07.2021, 21:26 [ТС]	3
	Сообщение от DrOffset squareroot, локаль скорее всего стоит русская. Проверить просто - заменить точку на запятую. Вряд ли баг. Вы были правы. Это невероятно даже о таком догадаться. Сообщение от DrOffset Зависит от особенностей вашего проекта. Абстрактно - ничего плохого в них нет, если использовать правильно. Первый минус я нашел - распознавание зависит от текущей локали. А какие еще сюрпризы мне ждать ? Там в основе вариативная функция, которая выглядит небезопасной и устаревшей с появлением вариативных шаблонов. Думаю std::getline + std::regex + конвертеры строка-число выиграют у sscanf по скорости, хотя надо мерить. 0

@avgoor 1550 / 875 / 179 Регистрация: 05.12.2015 Сообщений: 2,555
	01.07.2021, 21:42	4
	Сообщение от squareroot Первый минус я нашел - распознавание зависит от текущей локали. Это не минус. Локаль для этого и предназначена. 0

@DrOffset 18840 / 9839 / 2408 Регистрация: 30.01.2014 Сообщений: 17,280
	01.07.2021, 22:24	6
	squareroot, это другое. В смысле у cout своя локаль, а scanf использует глобальную. Добавлено через 2 минуты Сообщение от squareroot Думаю std::getline + std::regex + конвертеры строка-число выиграют у sscanf по скорости Очень вряд ли. Сообщение от squareroot Там в основе вариативная функция, которая выглядит небезопасной и устаревшей с появлением вариативных шаблонов. Переубеждать тут вас никто не будет, не подходит, значит не подходит. Дело ваше же. 0

@DrOffset 18840 / 9839 / 2408 Регистрация: 30.01.2014 Сообщений: 17,280
	01.07.2021, 22:45	8
	Сообщение от squareroot то тоже мимо Ну вот из-за таких вот высказываний и не хочется совершено ни о чем дискутировать. Это не глобальная локаль, а временный объект локали. Глобальная - это вот: https://en.cppreference.com/w/... ale/global 0

@squareroot 13 / 13 / 1 Регистрация: 19.10.2019 Сообщений: 607
	01.07.2021, 22:49 [ТС]	9
	Сообщение от DrOffset Ну вот из-за таких вот высказываний и не хочется совершено ни о чем дискутировать. Это не глобальная локаль, а временный объект локали. Глобальная - это вот: https://en.cppreference.com/w/... ale/global Я смотрел эту функцию. Она нужна чтоб менять локаль, А не получать. Так сказано в документации. А то что я привел, то там черным по белому написано: Default constructor. Constructs a copy of the global C++ locale 0

@DrOffset 18840 / 9839 / 2408 Регистрация: 30.01.2014 Сообщений: 17,280
	01.07.2021, 22:52	10
	Сообщение от squareroot А то что я привел, то там черным по белому написано Вот именно, вы сами-то прочитайте что там написано. Создает копию глобальной локали. Копию! Еще раз, не создает глобальную локаль, а создает копию глобальной локали. 0

@DrOffset 18840 / 9839 / 2408 Регистрация: 30.01.2014 Сообщений: 17,280
	01.07.2021, 22:57	12
	Сообщение от squareroot А не получать. Зачем ее получать-то? Чтобы повлиять на поведение функции, нужно наоборот ее задать. Во-вторых, если вы уже читаете документацию, то там как бы четко написано, что в программе на С++ две локали, С и С++. Это разные вещи. Совпадать они начнут только после вызова функции `global`. Добавлено через 1 минуту Сообщение от squareroot Кто по копии мне узнать мешает ? Вы по копии С++ локали хотите узнать настройки локали С? Я такого сразу предположить не смог, извините. 0

@squareroot 13 / 13 / 1 Регистрация: 19.10.2019 Сообщений: 607
	01.07.2021, 23:02 [ТС]	13
	DrOffset, Ой, ну и заморочек с этой локалью. Ок. Буду искать как вытащить запятую/точку из Си локали. 0

@DrOffset 18840 / 9839 / 2408 Регистрация: 30.01.2014 Сообщений: 17,280
	01.07.2021, 23:05	14
	Сообщение от squareroot Буду искать как вытащить запятую/точку из Си локали. https://en.cppreference.com/w/cpp/locale/lconv 0

@squareroot 13 / 13 / 1 Регистрация: 19.10.2019 Сообщений: 607
	02.07.2021, 14:24 [ТС]	16
	DrOffset, А может sscanf приводить к сбоям в работе при некорректной строке ? Надо ли предварительно проверять строку регуляркой ? Добавлено через 5 минут Вот даже в инете есть коечто на этот счет: https://stackoverflow.com/ques... rogramming Добавлено через 22 минуты И Википедия тоже не молчит: scanf is vulnerable to format string attacks. Great care should be taken to ensure that the formatting string includes limitations for string and array sizes. In most cases the input string size from a user is arbitrary and cannot be determined before the scanf function is executed. This means that uses of %s placeholders without length specifiers are inherently insecure and exploitable for buffer overflows. Another potential problem is to allow dynamic formatting strings, for example formatting strings stored in configuration files or other user-controlled files. In this case the allowed input length of string sizes cannot be specified unless the formatting string is checked beforehand and limitations are enforced. Related to this are additional or mismatched formatting placeholders which do not match the actual vararg list. These placeholders might be partially extracted from the stack or contain undesirable or even insecure pointers, depending on the particular implementation of varargs. Добавлено через 5 часов 24 минуты Интересная ситуювина. А sscanf_s под линукс нету ? Добавлено через 7 минут макроопределение добавил #define __STDC_WANT_LIB_EXT1__ 1 0

@DrOffset 18840 / 9839 / 2408 Регистрация: 30.01.2014 Сообщений: 17,280
	02.07.2021, 16:14	19
	Сообщение от squareroot . Грубо говоря, подали что то совсем нечитабельное(какой бинарный код например) Это общая проблема, она не касается конкретно scanf. Scanf обрубит до первого нуля этот бинарный код, если это как строка подавалось и все. Вообще любой пользовательский ввод, если это важно, нужно защищать от такого. Добавлено через 15 секунд Сообщение от squareroot 1) А где 2) ? 0

@squareroot 13 / 13 / 1 Регистрация: 19.10.2019 Сообщений: 607
	02.07.2021, 16:20 [ТС]	20
	Сообщение от DrOffset Это общая проблема, она не касается конкретно scanf. Scanf обрубит до первого нуля этот бинарный код, если это как строка подавалось и все. Вообще любой пользовательский ввод, если это важно, нужно защищать от такого. Вот я и хотел знать надо ли защищать. В данном случае скорее не пользовательский ввод, а процес чтения файла, где может появится чтото непотребное. И не будет ли лишней это ? Ведь sscanf тоже что то проверяет. А проверка регуляркой недешевая операция. Сообщение от DrOffset Добавлено через 15 секунд А где 2) ? Это оно и есть. 0

Ошибка ввода float чисел

Решение