我在C中有以下功能:
char* AUDIO_GetFile(FIL* fil, DIR* dir, FILINFO* fno) {
FRESULT res;
res = f_readdir(dir, fno); /* Read a directory item */
if (fno->fname[0] == '\0')} /* End of files */
...
}
...
return &(fno->fname);
}
但是我需要一个临时变量来保留旧值然后返回&(fno->fname)
。我创建了局部变量FILINFO t_fno
,然后将其传递给函数res = f_readdir(dir, &t_fno);
一切正常。
但是,如果我想将t_fno用作本地指针,然后将其传递给f_readdir,我就不会获得实际数据,但会在if (t_fno->fname[0] == '\0')}
上获得一些随机内存读数和程序崩溃。我也尝试了FILINFO* t_fno = NULL
,但这并没有帮助。
char* AUDIO_GetFile(FIL* fil, DIR* dir, FILINFO* fno) {
FRESULT res;
FILINFO* t_fno;
res = f_readdir(dir, t_fno); /* Read a directory item */
if (t_fno->fname[0] == '\0')} /* crash */
...
}
...
fno = t_fno;
return &(fno->fname);
}
FRESULT f_readdir (
DIR* dp, /* Pointer to the open directory object */
FILINFO* fno /* Pointer to file information to return */
)
{
FRESULT res;
DEFINE_NAMEBUF;
res = validate(dp); /* Check validity of the object */
if (res == FR_OK) {
if (!fno) {
res = dir_sdi(dp, 0); /* Rewind the directory object */
} else {
INIT_BUF(*dp);
res = dir_read(dp, 0); /* Read an item */
if (res == FR_NO_FILE) { /* Reached end of directory */
dp->sect = 0;
res = FR_OK;
}
if (res == FR_OK) { /* A valid entry is found */
get_fileinfo(dp, fno); /* Get the object information */
res = dir_next(dp, 0); /* Increment index for next */
if (res == FR_NO_FILE) {
dp->sect = 0;
res = FR_OK;
}
}
FREE_BUF();
}
}
LEAVE_FF(dp->fs, res);
}
答案 0 :(得分:1)
根据its docs,f_readdir()
期望其第二个参数是文件信息结构的" [p] ointer,用于存储有关读取项目的信息。"这与您描述的第一次使用一致,其中您将t_fno
声明为本地FILINFO
结构,并传递其地址。 不与将t_fno
声明为指针,保持未初始化并传递其不确定的初始值一致。
第二种方式产生未定义的行为,这似乎是非常合理的,因为最可能的慈善解释是你已经指示函数在某个随机位置写入数据。即使成功这样做,您也无法知道效果是什么,或者当您稍后尝试读取数据时数据是否会保持不变,或者是否可以稍后再读取它们。 C不要求或提供任何理由 - 它只是声明行为是未定义的。你无法安全或可靠地完成这项工作。
正如您所观察到的,初始化指针并没有帮助,除非(您没有探索),您将其初始化为指向与FILINFO
结构兼容的实际存储。指针声明本身仅为指针提供,而不是指向任何指针。例如:
char* AUDIO_GetFile(FIL* fil, DIR* dir, FILINFO* fno) {
FRESULT res;
FILINFO t_info;
FILINFO* t_fno = &t_info;
res = f_readdir(dir, t_fno); /* Read a directory item */
// ...
在大多数情况下,我不想单独维护t_fno
,但保留它可能有可行的理由。