Skip to content

FIX handling of CAN device interface names #1715

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
djchhp wants to merge 2 commits into
base: main
Choose a base branch
from

Update socket_can_handler.cpp

f06b07f
Select commit
Loading
Failed to load commit list.
Open

FIX handling of CAN device interface names #1715

Update socket_can_handler.cpp
f06b07f
Select commit
Loading
Failed to load commit list.
Codacy Production / Codacy Static Code Analysis required action Jan 14, 2026 in 0s

5 new issues (0 max.) of at least minor severity.

Codacy Here is an overview of what got changed by this pull request:

Issues
======
- Added 5
           

Complexity increasing per file
==============================
- modules/HardwareDrivers/PowerSupplies/UUGreenPower_UR1000X0/can_driver_acdc/CanDevice.cpp  3
- modules/HardwareDrivers/PowerSupplies/InfyPower_BEG1K075G/can_driver_acdc/CanDevice.cpp  3
- lib/everest/io/src/can/socket_can_handler.cpp  1

See the complete overview on Codacy

Annotations

Check failure on line 110 in lib/everest/io/src/can/socket_can_handler.cpp

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

lib/everest/io/src/can/socket_can_handler.cpp#L110 

Easily used incorrectly; doesn't always \0-terminate or check for invalid pointers [MS-banned] (CWE-120).

Check failure on line 32 in modules/HardwareDrivers/PowerSupplies/InfyPower_BEG1K075G/can_driver_acdc/CanDevice.cpp

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

modules/HardwareDrivers/PowerSupplies/InfyPower_BEG1K075G/can_driver_acdc/CanDevice.cpp#L32 

Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126).

Check failure on line 45 in modules/HardwareDrivers/PowerSupplies/InfyPower_BEG1K075G/can_driver_acdc/CanDevice.cpp

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

modules/HardwareDrivers/PowerSupplies/InfyPower_BEG1K075G/can_driver_acdc/CanDevice.cpp#L45 

Easily used incorrectly; doesn't always \0-terminate or check for invalid pointers [MS-banned] (CWE-120).

Check failure on line 32 in modules/HardwareDrivers/PowerSupplies/UUGreenPower_UR1000X0/can_driver_acdc/CanDevice.cpp

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

modules/HardwareDrivers/PowerSupplies/UUGreenPower_UR1000X0/can_driver_acdc/CanDevice.cpp#L32 

Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126).

Check failure on line 45 in modules/HardwareDrivers/PowerSupplies/UUGreenPower_UR1000X0/can_driver_acdc/CanDevice.cpp

See this annotation in the file changed.

@codacy-production codacy-production / Codacy Static Code Analysis

modules/HardwareDrivers/PowerSupplies/UUGreenPower_UR1000X0/can_driver_acdc/CanDevice.cpp#L45 

Easily used incorrectly; doesn't always \0-terminate or check for invalid pointers [MS-banned] (CWE-120).
View more details on Codacy Production