fix(vfs): fix stat initialization, close espressif#11908 (espressif#11911)

JakubAndrysek · web-flow · commit 34d9855291c0 · 2025-10-17T13:15:44.000+03:00
* fix(vfs): fix stat initialisation, close espressif#11908 * fix(vfs): change log level from warning to error for stat failure
diff --git a/libraries/FS/src/vfs_api.cpp b/libraries/FS/src/vfs_api.cpp
@@ -276,8 +276,12 @@ VFSFileImpl::VFSFileImpl(VFSImpl *fs, const char *fpath, const char *mode) : _fs
     if (!_f) {
       log_e("fopen(%s) failed", temp);
     }
-    if (_f && (_stat.st_blksize == 0)) {
-      setvbuf(_f, NULL, _IOFBF, DEFAULT_FILE_BUFFER_SIZE);
+    if (!stat(temp, &_stat)) {
+      if (_f && (_stat.st_blksize == 0)) {
+        setvbuf(_f, NULL, _IOFBF, DEFAULT_FILE_BUFFER_SIZE);
+      }
+    } else {
+      log_e("stat(%s) failed", temp);
     }
   }
   free(temp);

Original file line number	Diff line number	Diff line change
`@@ -276,8 +276,12 @@ VFSFileImpl::VFSFileImpl(VFSImpl fs, const char fpath, const char *mode) : _fs`
`276`	`276`	`if (!_f) {`
`277`	`277`	`log_e("fopen(%s) failed", temp);`
`278`	`278`	`}`
`279`		`- if (_f && (_stat.st_blksize == 0)) {`
`280`		`- setvbuf(_f, NULL, _IOFBF, DEFAULT_FILE_BUFFER_SIZE);`
	`279`	`+ if (!stat(temp, &_stat)) {`
	`280`	`+ if (_f && (_stat.st_blksize == 0)) {`
	`281`	`+ setvbuf(_f, NULL, _IOFBF, DEFAULT_FILE_BUFFER_SIZE);`
	`282`	`+ }`
	`283`	`+ } else {`
	`284`	`+ log_e("stat(%s) failed", temp);`
`281`	`285`	`}`
`282`	`286`	`}`
`283`	`287`	`free(temp);`