Description
An issue was discovered in OpenSatKit 2.2.1. The DirName field in the telecommand is provided by the ground segment and must be treated as untrusted input. The program copies DirName into the local buffer DirWithSep using strcpy. The size of this buffer is OS_MAX_PATH_LEN. If the length of DirName is greater than or equal to OS_MAX_PATH_LEN, a stack buffer overflow occurs, overwriting adjacent stack memory. The path length check (FileUtil_AppendPathSep) is performed after the strcpy operation, meaning the validation occurs too late and cannot prevent the overflow.
References
github.com/OpenSatKit/OpenSatKit
github.com/OpenSatKit/OpenSatKit/releases/tag/v2.2.1
raw.githubusercontent.com/.../cfs/apps/filemgr/fsw/src/dir.c
raw.githubusercontent.com/.../cfs/apps/filemgr/fsw/src/dir.c