Home / reactos / reactos
Repository Analysis

reactos/reactos

A free Windows-compatible Operating System

1.8 Likely human-written View on GitHub
1.8
Adjusted Score
1.8
Raw Score
100%
Time Factor
2026-05-30
Last Push
17,575
Stars
C
Language
9,320,974
Lines of Code
17943
Files
18649
Pattern Hits
2026-05-31
Scan Date

Score History

Severity Breakdown

CRITICAL 2HIGH 2MEDIUM 32LOW 18613

Pattern Findings

18649 matches across 14 categories. Click a row to expand file-level details.

Over-Commented Block18538 hits · 16499 pts
SeverityFileLineSnippet
LOWdrivers/filters/fltmgr/Object.c21/* DATA *********************************************************************/
LOWdrivers/filters/fltmgr/fltmgr.h1#ifndef _FLTMGR_H
LOWdrivers/filters/fltmgr/fltmgr.h121
LOWdrivers/filters/fltmgr/fltmgr.h141//FMil - fltmgr.sys - IRP_CTRL completion node stack
LOWdrivers/acpi/cmbatt/cmbatt.h41#define CMBATT_ACPI_ENTRY_EXIT 0x40
LOWdrivers/acpi/cmbatt/cmbatt.h61} CMBATT_EXTENSION_TYPE;
LOWdrivers/parallel/parallel/parallel.h61#define LPGETIRQ 0x0606 /* get the current IRQ number */
LOWdrivers/sac/driver/sacdrv.h21//
LOWdrivers/sac/driver/sacdrv.h121 KeClearEvent(Channel->x##WaitObjectBody); \
LOWdrivers/sac/driver/sacdrv.h141#define FREE_POOL_TAG 'FpcR'
LOWdrivers/sac/driver/sacdrv.h161#define SAC_VTUTF8_OBUFFER_SIZE 0x2D00
LOWdrivers/sac/driver/sacdrv.h1321{
LOWdrivers/sac/driver/sacdrv.h1481// The following site is a good reference on VT100/ANSI escape codes
LOWdrivers/sac/driver/sacdrv.h1501#define VT_ANSI_ERASE_START_LINE L"[1K"
LOWdrivers/input/inport/hardware.c21
LOWdrivers/input/inport/hardware.c41
LOWdrivers/input/inport/hardware.c61 * Microsoft InPort
LOWdrivers/input/inport/hardware.c81
LOWdrivers/network/netio/netio.c41#include <ntifs.h>
LOWdrivers/network/tdi/precomp.h1#ifndef _TDI_PCH_
LOWdrivers/network/afd/include/debug.h41 : (DebugTraceLevel & NORMAL_MASK) >= _t_) { \
LOWdrivers/network/afd/include/afd.h21#include <windef.h>
LOWdrivers/network/afd/include/afd.h41#define TAG_AFD_CONNECT_DATA 'cdfA'
LOWdrivers/network/afd/include/afd.h81#define SOCKET_STATE_LOCKED 0x40000000
LOWdrivers/network/dd/nvnet/nvnet.h21
LOWdrivers/network/dd/nvnet/nvnet.h41#define NVNET_RECEIVE_PROCESSING_LIMIT 64
LOWdrivers/network/dd/nvnet/nvnet.h281{
LOWdrivers/network/dd/nvnet/nic.h21 * and needs packet format 2 */
LOWdrivers/network/dd/nvnet/nic.h41#define DEV_NEED_PHY_INIT_FIX 0x00400000 /* Device needs specific PHY workaround */
LOWdrivers/network/dd/nvnet/nic.h61#define NVREG_IRQ_TX_OK 0x0010
LOWdrivers/network/dd/nvnet/nic.h101#define NVREG_XMITCTL_MGMT_SEMA_MASK 0x00000f00
LOWdrivers/network/dd/nvnet/nic.h121 NvRegOffloadConfig = 0x90,
LOWdrivers/network/dd/nvnet/nic.h181#define NVREG_RINGSZ_RXSHIFT 16
LOWdrivers/network/dd/nvnet/nic.h201 NvRegTxRxControl = 0x144,
LOWdrivers/network/dd/nvnet/nic.h221#define NVREG_TX_PAUSEFRAME_ENABLE_V3 0x09f00880
LOWdrivers/network/dd/nvnet/nic.h281#define NVREG_MGMTUNITVERSION 0x08
LOWdrivers/network/dd/nvnet/nic.h361typedef struct _NVNET_DESCRIPTOR_64
LOWdrivers/network/dd/nvnet/nic.h381#define NV_TX_CARRIERLOST (1<<27)
LOWdrivers/network/dd/nvnet/nic.h401#define NV_TX2_CHECKSUM_L3 (1<<27)
LOWdrivers/network/dd/nvnet/nic.h421 NV_RX_OVERFLOW|NV_RX_FRAMINGERR)
LOWdrivers/network/dd/nvnet/nic.h441
LOWdrivers/network/dd/nvnet/nic.h461#define NV_MIIBUSY_DELAY 50
LOWdrivers/network/dd/nvnet/nic.h481#define PHY_MODEL_REALTEK_8211 0x0110
LOWdrivers/network/dd/nvnet/nic.h501#define PHY_CICADA_INIT5 0x0004
LOWdrivers/network/dd/nvnet/nic.h521#define PHY_REALTEK_INIT11 0x0200
LOWdrivers/network/dd/nvnet/nic.h541#define PHY_REALTEK_INIT3 0x0001
LOWdrivers/network/dd/nvnet/nic.h561#define NV_MSI_X_VECTOR_TX 0x1
LOWdrivers/network/dd/nvnet/phyreg.h1/*
LOWdrivers/network/dd/nvnet/phyreg.h21#define MII_AUTONEG_ADVERTISE 0x04
LOWdrivers/network/dd/nvnet/phyreg.h41#define MII_MASTER_SLAVE_STATUS 0x0A
LOWdrivers/network/dd/dc21x4/eeprom.h41#define SRomGetExtendedBlockLength(Byte) (((Byte) & 0x7F) + 1)
LOWdrivers/network/dd/dc21x4/eeprom.h61
LOWdrivers/network/dd/dc21x4/eeprom.h81/*
LOWdrivers/network/dd/dc21x4/eeprom.h101#define SROM_MEDIA_100T_HD 3
LOWdrivers/network/dd/dc21x4/debug.h81
LOWdrivers/network/dd/dc21x4/media.h41} DC_MII_MEDIA, *PDC_MII_MEDIA;
LOWdrivers/network/dd/dc21x4/dc21x4hw.h21 */
LOWdrivers/network/dd/dc21x4/dc21x4hw.h61#define DC_TBD_STATUS_UNDERFLOW 0x00000002
LOWdrivers/network/dd/dc21x4/dc21x4hw.h81#define DC_TBD_CONTROL_CHAINED 0x01000000
LOWdrivers/network/dd/dc21x4/dc21x4hw.h101/*
18478 more matches not shown…
AI Slop Vocabulary24 hits · 72 pts
SeverityFileLineSnippet
LOWdrivers/network/tcpip/lwip/src/include/lwip/sockets.h214#define SO_DONTROUTE 0x0010 /* Unimplemented: just use interface addresses */
MEDIUMdrivers/storage/partmgr/partition.c865 // so we just return success. A more robust algorithm would be
MEDIUMdrivers/filesystems/udfs/write.cpp749 FALSE, // We will not utilize pin access for this file
MEDIUMdrivers/filesystems/udfs/read.cpp640 FALSE, // We will not utilize pin access for this file
MEDIUMdrivers/filesystems/udfs/read.cpp768 // Also, at this point, the FSD might choose to utilize the
MEDIUMdrivers/filesystems/fastfat/easup.c3232 // nice and more robust
MEDIUMdrivers/filesystems/fastfat/close.c674 // make sure we fully utilize the acquisition of the volume, which can
MEDIUMsdk/include/c++/stlport/stl/config/_apple.h97#define _STLP_VENDOR_GLOBAL_STD // mpw's c++ libs do not utilize namespace std yet
MEDIUMsdk/include/c++/stlport/stl/config/_apple.h97#define _STLP_VENDOR_GLOBAL_STD // mpw's c++ libs do not utilize namespace std yet
MEDIUMsdk/include/psdk/twain.h1311#define DG_CONTROL 0x0001L /* data pertaining to control */
MEDIUMsdk/include/psdk/twain.h1312#define DG_IMAGE 0x0002L /* data pertaining to raster images */
MEDIUMsdk/include/psdk/twain.h1314#define DG_AUDIO 0x0004L /* data pertaining to audio */
MEDIUMsdk/include/ucrt/corecrt.h89// C28719: Banned API, use a more robust and secure replacement.
MEDIUMsdk/include/ucrt/corecrt.h90// C28726: Banned or deprecated API, use a more robust and secure replacement.
MEDIUM…drivers/wdf/shared/inc/private/common/fxdeviceinit.hpp247 // If set, the Cx/Client intends to leverage Self IO Target
MEDIUM…ib/drivers/wdf/shared/inc/private/common/fxioqueue.hpp541 // The IoQueue is designed to be robust in that multiple events
MEDIUMsdk/lib/drivers/wdf/shared/targets/usb/fxusbdevice.cpp1535 // but can't leverage on that as that sends the select interface URB down
MEDIUMsdk/lib/ucrt/stdio/fopen.cpp38 // subjected to direct user input and thus must be robust to that rather
MEDIUMsdk/lib/ucrt/heap/debug_heap.cpp1772// alignment.cpp, but these functions (suffixed with _dbg instead of _base) utilize
MEDIUMsubsystems/mvdm/ntvdm/bios/vidbios.c3351 // Certainly in BL and not in BX as said by Ralf Brown...
MEDIUMsubsystems/mvdm/ntvdm/bios/vidbios.c3369 // Certainly in BL and not in BX as said by Ralf Brown...
MEDIUMsubsystems/mvdm/ntvdm/bios/vidbios.c3429 // Certainly in BL and not in BX as said by Ralf Brown...
MEDIUMwin32ss/drivers/displays/vga/vgaddi.h188 // one seamless 64K bank
LOW…s/applications/devutils/btrfstools/btrfs_structures.py889 # sizes, because that's how it gets cowed. We don't just use 1 or 2
Slop Phrases26 hits · 41 pts
SeverityFileLineSnippet
LOWdrivers/network/tcpip/lwip/src/core/netif.c280 * IPv6: Don't forget to call netif_create_ip6_linklocal_address() after
LOWdrivers/storage/mountmgr/device.c1407 /* And don't forget to nullify */
LOWdrivers/storage/port/scsiport/scsi.c1292 /* Nothing - just return (don't forget to release the spinlock */
LOWdrivers/multimedia/audio/sndblst.old/dma.c11 // Don't forget to check for Compaq machines (they can't be configured
LOWsdk/include/c++/stlport/stl/config/user_config.h217 * If you customize the STLport generated library names don't forget to give
LOWsdk/include/reactos/libs/mbedtls/blowfish.h67#define MBEDTLS_BLOWFISH_ROUNDS 16 /**< Rounds to use. When increasing this value, make sure to extend the
LOWsdk/lib/drivers/csq/csq.c45 * - Don't forget to release the cancel spinlock ASAP --> #1 hot lock in the
LOWsdk/lib/rtl/heap.c1937 /* Don't forget to update previous size of the next entry! */
MEDIUMsdk/lib/3rdparty/freetype/include/freetype/fterrors.h78 * Here is a simple example.
MEDIUMsdk/lib/3rdparty/freetype/include/freetype/freetype.h1182 * for demonstration purposes.
LOW…/3rdparty/freetype/include/freetype/internal/ftrfork.h35 /* Don't forget to increment the number if you add a new guessing rule. */
LOWntoskrnl/fsrtl/oplock.c351 /* Don't forget to release the mutex */
LOWntoskrnl/cc/copy.c370 /* And don't forget to release master */
LOWdll/win32/cabinet/fdi.c2378 * This notification is important, don't forget to implement it. This
LOWdll/win32/advapi32/sec/misc.c480 * more than 0xFFFF (USHORT) characters! Also don't forget to explicitly
LOWdll/win32/browseui/travellog.cpp505 // don't forget to patch ampersands before adding to menu
LOWdll/directx/wine/ddraw/utils.c1222 * though. Don't forget to set ddsCaps.dwCaps2/3/4 to 0 when removing this */
LOWdll/directx/wine/wined3d/arb_program_shader.c7077 * Don't forget to clamp the y values in into the range, otherwise we'll get filtering
LOWdll/opengl/glu32/src/libutil/mipmap.c4536 free(newMipmapImage); /* don't forget to free it! */
LOWwin32ss/user/ntuser/class.c2979// Don't forget to use DesktopPtrToUser( ? ) with return pointer in user space.
MEDIUMmodules/rosapps/applications/net/ncftp/sio/sio.html738<P>This is <TT><A HREF="#SSendto">SSendto()</A></TT>, only you can use
LOWmodules/rosapps/applications/net/ncftp/libncftp/cmds.c1157 /* Don't forget to change back to the original directory. */
LOWmodules/rosapps/applications/net/ncftp/libncftp/cmds.c1305 /* Don't forget to change back to the original directory. */
LOWbase/applications/regedit/framewnd.c740 pd.hDevMode = NULL; /* Don't forget to free or store hDevMode*/
LOWbase/applications/regedit/framewnd.c741 pd.hDevNames = NULL; /* Don't forget to free or store hDevNames*/
LOWbase/services/nfsd/nfs41_daemon.c53#ifndef STANDALONE_NFSD //make sure to define it in "sources" not here
Hallucination Indicators2 hits · 30 pts
SeverityFileLineSnippet
CRITICALdrivers/network/tcpip/lwip/src/apps/snmp/snmp_core.c258 * device a > 1.3.6.1.4.1.XXX(ent-oid).1(devices).1(device a)
CRITICALdrivers/network/tcpip/lwip/src/apps/snmp/snmp_core.c259 * device b > 1.3.6.1.4.1.XXX(ent-oid).1(devices).2(device b)
Verbosity Indicators15 hits · 22 pts
SeverityFileLineSnippet
LOWdrivers/network/tcpip/lwip/src/apps/mdns/mdns.c614 * comparison. So if we get passed the class and type comparison we need to check if the
LOWdrivers/bus/acpi/acpica/executer/exmisc.c209 * We need to check if the shiftcount is larger than the integer bit
LOWdrivers/bus/acpi/acpica/executer/exmisc.c221 * We need to check if the shiftcount is larger than the integer bit
LOWdrivers/storage/class/cdrom/cdrom.c1085 The purpose of this function is to send SYNC CACHE command and STOP UNIT command if it's necessary.
LOWdrivers/filesystems/cdfs/dirsup.c241 find. This may already point to a dirent so we need to check if
LOWsdk/lib/cportlib/cport_pc98.c439 * To avoid an infinite loop, we need to check if the other side is ready
LOWsdk/lib/ucrt/internal/locks.cpp16// This variable stores the number of locks that have been successfully
LOWntoskrnl/io/pnpmgr/devaction.c1108 // Step 1: Write the DeviceDesc value if does not exist
LOWntoskrnl/io/pnpmgr/devaction.c1157 // Step 2: LocaltionInformation is overwritten unconditionally
LOWdll/win32/shell32/CDefView.cpp4349 /* We need to check if we drag the selection over itself */
LOWwin32ss/user/ntuser/security.c18 * context of the caller. The purpose of this function is to query
LOWbase/setup/lib/utils/partlist.c3336// Step 1 : Check the system disk.
LOWbase/setup/lib/utils/partlist.c3456// Step 2 : No active partition found: Check the alternative disk if specified.
LOWbase/services/umpnpmgr/install.c533 // Step 1: install all drivers which were configured during the boot
LOWbase/services/umpnpmgr/install.c580 // Step 2: start the wait-loop for newly added devices
Fake / Example Data11 hits · 16 pts
SeverityFileLineSnippet
LOWmodules/rostests/winetests/gdi32/font.c2377 "Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do "
LOWmodules/rostests/winetests/gdi32/font.c2377 "Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do "
LOWmodules/rostests/winetests/riched20/txtsrv.c124static const WCHAR lorem[] = L"Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt
LOWmodules/rostests/winetests/riched20/txtsrv.c124static const WCHAR lorem[] = L"Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt
LOWmodules/rostests/winetests/riched20/editor.c7306 result = SendMessageA(hwnd, WM_SETTEXT, 0, (LPARAM)"Lorem ipsum\rdolor sit\rnamet");
LOWmodules/rostests/winetests/riched20/editor.c7309 /* [|Lorem ipsum] [dolor sit] [amet] */
LOWmodules/rostests/winetests/riched20/editor.c7319 /* [Lorem ipsum|] [dolor sit] [amet] */
LOWmodules/rostests/winetests/riched20/editor.c7326 /* [Lorem ipsum] [|dolor sit] [amet] */
LOWmodules/rostests/winetests/riched20/editor.c7333 /* [Lorem ipsum|] [dolor sit] [amet] */
LOWmodules/rostests/apitests/winspool/EnumPrinters.c129 ok(!EnumPrintersW(PRINTER_ENUM_LOCAL | PRINTER_ENUM_NAME, L"LOREM IPSUM", Level, NULL, 0, &cbNeeded, &dw
LOWmodules/rostests/apitests/winspool/EnumPrinters.c168 ok(!EnumPrintersW(PRINTER_ENUM_LOCAL, L"LOREM IPSUM", Level, NULL, 0, &cbNeeded, &dwReturned), "EnumPrinters
Synthetic Comment Markers2 hits · 15 pts
SeverityFileLineSnippet
HIGHdrivers/storage/class/sfloppy/floppy.c3061 // If we don't get as much data as requested, it is not an error.
HIGHntoskrnl/mm/ARM3/virtual.c5080 // And now write the invalid demand-zero PTE as requested
Self-Referential Comments5 hits · 14 pts
SeverityFileLineSnippet
MEDIUMdrivers/network/tcpip/lwip/codespell_changed_files.sh6# This file is subject to the terms and conditions of the GNU Lesser
MEDIUMdrivers/network/tcpip/lwip/codespell_check.sh5# This file is subject to the terms and conditions of the GNU Lesser
MEDIUMboot/remaster.sh157## Create a mkisofs sort file to specify an explicit ordering for the boot files
MEDIUMmodules/rosapps/applications/net/ncftp/ncftp/pref.c715 (void) fprintf(fp, "%s", "# NcFTP 3 preferences file\n# This file is loaded and overwritten each time NcFTP is run.\n#
MEDIUM…les/rosapps/applications/devutils/btrfstools/crc32c.py3# This file is part of the python-btrfs module.
Unused Imports9 hits · 9 pts
SeverityFileLineSnippet
LOWsdk/tools/gen_baseaddress.py8
LOWsdk/tools/gen_baseaddress.py8
LOWsdk/tools/gen_baseaddress.py8
LOWsdk/tools/winesync/winesync.py3
LOWsdk/tools/winesync/winesync.py9
LOW…s/applications/devutils/btrfstools/btrfs_playground.py6
LOW…s/applications/devutils/btrfstools/btrfs_structures.py8
LOW…s/applications/devutils/btrfstools/btrfs_structures.py14
LOW…s/applications/devutils/btrfstools/btrfs_structures.py16
Deep Nesting7 hits · 7 pts
SeverityFileLineSnippet
LOWsdk/tools/check_packing.py34
LOWsdk/tools/gen_baseaddress.py353
LOWsdk/tools/update_caroots.py81
LOWsdk/tools/winesync/winesync.py128
LOW…s/applications/devutils/btrfstools/btrfs_structures.py921
LOW…s/applications/devutils/btrfstools/btrfs_structures.py1160
LOW…s/applications/devutils/btrfstools/btrfs_structures.py1252
Hyper-Verbose Identifiers6 hits · 6 pts
SeverityFileLineSnippet
LOWdrivers/bus/acpi_new/uacpi/include/uacpi/tables.h117typedef uacpi_table_installation_disposition (*uacpi_table_installation_handler)
LOWsdk/tools/winesync/winesync.py13def string_to_valid_file_name(to_convert):
LOWsdk/tools/winesync/winesync.py63 def create_or_checkout_wine_branch(self, wine_tag, wine_staging_tag):
LOWdll/win32/ole32/clipboard.c1659 * function IsClipboardFormatAvailable() otherwise.
LOWmodules/rosapps/applications/net/netreg/netreg.cpp42class RequestHandler {
LOWmodules/rostests/winetests/mshtml/events.html29 function ondataavailableDocHandler() {
Excessive Try-Catch Wrapping2 hits · 4 pts
SeverityFileLineSnippet
MEDIUMsdk/tools/winesync/winesync.py297 print(f'Error while reverting patch {patch_file_name}')
MEDIUMmodules/rostests/apitests/crt/gen_math_tests.py287 print(f"Error: Unsupported function '{function_name}'. Available tables: {', '.join(TABLE_FUNCTIONS.keys())}")
Example Usage Blocks1 hit · 2 pts
SeverityFileLineSnippet
LOWboot/remaster.sh51## Usage example:
Redundant / Tautological Comments1 hit · 2 pts
SeverityFileLineSnippet
LOWmodules/rostests/apitests/crt/gen_math_tests.py282 # Check if the table name is valid