Projekt Curl har slutat att använda funktionen strcpy i koden

Daniel Stenberg, skaparen av verktyget för att hämta och skicka data över nätet curl, har meddelat att användningen av funktionen strcpy() i projektets kodbas har upphört och att denna funktion förbjöds i framtiden. Beslutet följer upp ett avskaffande av funktionen strncpy() förra året, som kopierade ett angivet antal byte från den ingående strängen. Användningen av strncpy() medförde risk för fel på grund av att nulltecknet i slutet av strängen kunde hoppas över eller att nollor lades till.

Anrop till strncpy() har ersatts med funktionen strcpy(), som föregicks av minnesallokering för målbufferten med hänsyn till storleken på den ursprungliga strängen, eller så gjordes en kontroll av överensstämmelsen mellan storleken på den ursprungliga strängen och målbufferten. Ingen ersättning med funktionen strlcpy() gjordes, eftersom det alltid behövdes kopiera hela strängen eller returnera ett fel.

Nu har alla anrop till strcpy() ersatts med den nya funktionen curlx_strcopy(dest, dsize, src, slen). Funktions curlx_strcopy() kräver att storleken på den ursprungliga och målbufferten anges med resultatet att målbufferten alltid måste vara större än den ursprungliga för att rymma nulltecknet i slutet av strängen, vilket tvingas läggas till av funktionen för att undvika att det hoppas över vid kopiering. Om storleken på målbufferten är större än noll, men inte tillräcklig för att kopiera den avsedda strängen, så läggs ett nollbyte till i början. void curlx_strcopy(char *dest, size_t dsize, const char *src, size_t slen) { DEBUGASSERT(slen < dsize); if(slen < dsize) { memcpy(dest, src, slen); dest[slen] = 0; } else if(dsize) dest[0] = 0; }

Ersättningen av strcpy() med curlx_strcopy() har gjorts eftersom det finns en risk för mänskliga fel, vilket kan leda till att koden delas upp med kontroller/minnesallokering och anropet av strcpy(), till exempel vid impulsivt överförande av endast en del av koden eller insättning av kod mellan kontrollen och anropet av strcpy(). Dessutom kommer upphörandet av användningen av strcpy att bidra till att eliminera flöden av falska sårbarhetsmeddelanden på grund av felaktiga åtgärder från AI-verktyg, som anser att förekomsten av strcpy() är en sårbarhet utan att ta hänsyn till kontrollerna i koden.

Källa: opennet.ru