Início / unix / Perguntas / 790003
Accepted
Jeremy
Asked: 2025-01-23 04:01:29 +0800 CST

Alterando todos os strcpy para strscpy

  • 772

Estou aprendendo sobre desenvolvimento do kernel Linux. Eu clonei para o branch estável do Linux. Eu assisti a um tutorial de Greg KH no youtube que explica que você pode usar o script checkpatch.pl para encontrar código insatisfatório de acordo com os padrões atuais. Eu sou apenas um iniciante e então não tenho nenhum arquivo em mente para testar. Eu decidi testar aquele que inicia tudo: main.c em init/:

./scripts/checkpatch.pl --file --terse init/main.c

A saída foi uma longa lista de diferentes coisas erradas com o arquivo main.c:

homie@vmi2410184:~/git/kernels/staging$ ./scripts/checkpatch.pl --file --terse init/main.c
Traceback (most recent call last):
  File "/home/homie/git/kernels/staging/scripts/spdxcheck.py", line 6, in <module>
    from ply import lex, yacc
ModuleNotFoundError: No module named 'ply'
init/main.c:3: WARNING: It's generally not useful to have the filename in the file
init/main.c:107: WARNING: Use #include <linux/io.h> instead of <asm/io.h>
init/main.c:110: WARNING: Use #include <linux/cacheflush.h> instead of <asm/cacheflush.h>
init/main.c:203: WARNING: Missing a blank line after declarations
init/main.c:208: WARNING: Block comments use a trailing */ on a separate line
init/main.c:401: WARNING: braces {} are not necessary for single statement blocks
init/main.c:469: WARNING: void function return statements are not generally useful
init/main.c:567: WARNING: Missing a blank line after declarations
init/main.c:579: WARNING: Missing a blank line after declarations
init/main.c:659: WARNING: Prefer strscpy over strcpy - see: https://github.com/KSPP/linux/issues/88
init/main.c:660: WARNING: Prefer strscpy over strcpy - see: https://github.com/KSPP/linux/issues/88
init/main.c:662: WARNING: Prefer strscpy over strcpy - see: https://github.com/KSPP/linux/issues/88
init/main.c:663: WARNING: Prefer strscpy over strcpy - see: https://github.com/KSPP/linux/issues/88
init/main.c:675: WARNING: Prefer strscpy over strcpy - see: https://github.com/KSPP/linux/issues/88
init/main.c:677: WARNING: Prefer strscpy over strcpy - see: https://github.com/KSPP/linux/issues/88
init/main.c:681: WARNING: Prefer strscpy over strcpy - see: https://github.com/KSPP/linux/issues/88
init/main.c:683: WARNING: Prefer strscpy over strcpy - see: https://github.com/KSPP/linux/issues/88
init/main.c:1158: WARNING: Prefer strscpy over strcpy - see: https://github.com/KSPP/linux/issues/88
init/main.c:1196: WARNING: Prefer using '"%s...", __func__' to using 'initcall_blacklist', this function's name, in a string
init/main.c:1212: WARNING: Prefer [subsystem eg: netdev]_dbg([subsystem]dev, ... then dev_dbg(dev, ... then pr_debug(...  to printk(KERN_DEBUG ...
init/main.c:1222: WARNING: Prefer [subsystem eg: netdev]_dbg([subsystem]dev, ... then dev_dbg(dev, ... then pr_debug(...  to printk(KERN_DEBUG ...
init/main.c:1299: ERROR: Use of const init definition must use __initconst
init/main.c:1339: WARNING: Possible unnecessary 'out of memory' message
init/main.c:1343: WARNING: Prefer strscpy over strcpy - see: https://github.com/KSPP/linux/issues/88
init/main.c:1529: WARNING: quoted string split across lines
total: 1 errors, 24 warnings, 1603 lines checked

Essa pode ser uma pergunta idiota. Por que isso não foi corrigido?

Um em particular que seria fácil de mudar é mudar todo strcpy para strscpy. Tenha em mente que strscpy leva um argumento a mais que strcpy.

linux

1 respostas

  1. Best Answer

    As práticas de desenvolvimento do kernel evoluem com o tempo, mas o kernel em si é grande demais para que essas alterações sejam aplicadas a ele rapidamente. checkpatch.plé usado para verificar patches propostos para o kernel, não o código existente — isso garante que o kernel melhore progressivamente, mas, como você descobriu, isso não ajuda com código que não é alterado com muita frequência.

    Como você pode ver pelo problema referenciado na saída , a migração para strscpyestá sendo ativamente trabalhada e os patches são mesclados regularmente. Um patch abordando isso para init/main.cprovavelmente seria bem-vindo; copie Kees Cook sobre isso para obter ajuda para revisá-lo, caso precise.

    Você ainda precisa garantir que as mudanças estejam corretas e valham a pena. Como há um objetivo declarado de eliminar strcpyo uso, substituí-lo tem uma chance razoável de ser aceito. Algumas das outras mudanças defendidas por checkpatch.pl, como estrutura de comentários, linhas em branco e chaves desnecessárias, provavelmente não serão aceitas, a menos que você esteja alterando as linhas afetadas por outros motivos (válidos).

    • 17

relate perguntas