La faille a été ajoutée par un commit de l’an dernier ajoutant une feature « tabpanel » https://github.com/vim/vim/commit/be5bd4d6292fddcc103091407792730aaa48cc48
La PR : https://github.com/vim/vim/pull/17263
Perso je trouve ce commit plutôt moche, mais je n’ai pas d’habitude de lire les sources de vim. D’autres dans la PR disent que ça n’a pas l’air d’avoir été suffisamment retravaillé avant inclusion.
Je serai méchant je dirai que vu l’étendue et la taille du patch (ça touche au nombre de colonnes en largeurs, qu’on retrouve partout dans le code avec une variables Columns, maintenant remplacée par une macro moche COLUMNS_WITHOUT_TPL(), qui est une soustraction, alors que les calculs sont explicites partout ailleurs), ça serait l’idéal pour y planquer une faille.
Enfin, je pense surtout que ça a été fait assisté d’un LLM (hypothèse gratuite) parce que c’est moche, et que c’est ironique qu’on utilise des LLM pour dénicher des failles introduites par des LLM...
# La faille dans vim
Posté par benoar . En réponse au journal Claude trouve une faille dans Vim ET Emacs. Évalué à 3.
La faille a été ajoutée par un commit de l’an dernier ajoutant une feature « tabpanel » https://github.com/vim/vim/commit/be5bd4d6292fddcc103091407792730aaa48cc48
La PR : https://github.com/vim/vim/pull/17263
Perso je trouve ce commit plutôt moche, mais je n’ai pas d’habitude de lire les sources de vim. D’autres dans la PR disent que ça n’a pas l’air d’avoir été suffisamment retravaillé avant inclusion.
Je serai méchant je dirai que vu l’étendue et la taille du patch (ça touche au nombre de colonnes en largeurs, qu’on retrouve partout dans le code avec une variables Columns, maintenant remplacée par une macro moche COLUMNS_WITHOUT_TPL(), qui est une soustraction, alors que les calculs sont explicites partout ailleurs), ça serait l’idéal pour y planquer une faille.
Enfin, je pense surtout que ça a été fait assisté d’un LLM (hypothèse gratuite) parce que c’est moche, et que c’est ironique qu’on utilise des LLM pour dénicher des failles introduites par des LLM...