[lvc-project] [PATCH 6.1] mm/damon: get rid of overlapping areas

[Марков Глеб Игоревич]

При взаимодействии с strsep() замена strncmp() на strcmp() имеет место быть в совокупности.

Прошу прощения, смотрел не ту версию strim(), предлагаю отправить новую версию лишь с 
изменением sscanf() на strim().

Вторник, Июнь 23, 2026 13:37 MSK, Fedor Pchelkin <pchelkin at ispras.ru> писал(а):

 
On Tue, 23. Jun 13:26, Марков Глеб Игоревич wrote:
> Замена strncmp() на менее (на первый взгляд) безопасный strcmp() гарантирует
> корректность, поскольку его значение явно определено и работа с изолированным токеном,
> имхо, является более безопасным вариантом, чем использовать внешний count при сравнении.

Зачем это делать в текущем патче? Это как-то относится к решаемой
проблеме?

> Альтернатива в виде strim() тоже отпадает, поскольку обрезается лишь значение после токена,
> а теоретические лидирующие пробелы не отпадают, а значит работа будет вестить с
> некорректным значением.

/**
* strim - Removes leading and trailing whitespace from @s.
* @s: The string to be stripped.
*
* Note that the first trailing whitespace is replaced with a %NUL-terminator
* in the given string @s. Returns a pointer to the first non-whitespace
* character in @s.
*/
char *strim(char *s)
{
size_t size;
char *end;

size = strlen(s);
if (!size)
return s;

end = s + size - 1;
while (end >= s && isspace(*end))
end--;
*(end + 1) = '\0';

return skip_spaces(s);
}
EXPORT_SYMBOL(strim);
 



 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://linuxtesting.org/pipermail/lvc-project/attachments/20260623/2d033e92/attachment.html>