fix(vfs): initialize local variables in Linux target block to prevent scope leakage

mirror of https://github.com/espressif/esp-idf.git synced 2026-04-27 19:13:21 +00:00

In CMake v2's recursive component evaluation model, when a component
triggers the inclusion of another component via idf_component_include(),
the child component's directory scope inherits variables from the
caller's scope chain through the idf_component_include() function scope
and add_subdirectory().

The vfs component's Linux target block used list(APPEND ...) without
first initializing the srcs, inc, and priv_inc variables. This was
harmless when vfs was evaluated before esp_stdio, but after commit
5fac0b7386 ("feat(console): Move IO initialization outside of the
console component"), esp_stdio became a real component for the Linux
target and started calling idf_component_include(vfs). When esp_stdio
is evaluated first, its srcs variable (containing stdio_port.c and
linux/esp_stdio_linux.c) leaks into vfs's scope, causing the build
to fail with "Cannot find source file: components/vfs/stdio_port.c".

Fix by explicitly initializing all local variables at the top of the
Linux target block before appending to them.

Signed-off-by: Frantisek Hrbata <frantisek.hrbata@espressif.com>

This commit is contained in:

Frantisek Hrbata

2026-03-26 16:09:27 +01:00

parent 7b1d576416

commit 0505ced81d

1 changed files with 3 additions and 0 deletions

									
										components/vfs/CMakeLists.txt
									
		+3
		
												View File
												
				@@ -2,6 +2,9 @@ idf_build_get_property(target IDF_TARGET)

				# On Linux, we only support a few features, hence this simple component registration

				if(${target} STREQUAL "linux")

				    set(srcs)

				    set(inc)

				    set(priv_inc)

				    list(APPEND inc include)

				    if(CONFIG_VFS_SUPPORT_IO)