sergiotarxz/nuttx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

fs: Avoid accessing filep fields if it is NULL & add DEBUGASSERTs

Browse Source
This commit is contained in:
Alan Carvalho de Assis 2023-01-19 12:17:28 -03:00 committed by Xiang Xiao
parent 125d090b9c
commit c3a34a5956
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
fs/inode/fs_files.c
View File

@ -269,6 +269,9 @@ int files_duplist(FAR struct filelist *plist, FAR struct filelist *clist)
int i; int i;
int j; int j;
DEBUGASSERT(clist);
DEBUGASSERT(plist);
ret = nxmutex_lock(&plist->fl_lock); ret = nxmutex_lock(&plist->fl_lock);
if (ret < 0) if (ret < 0)
{ {
@ -298,7 +301,10 @@ int files_duplist(FAR struct filelist *plist, FAR struct filelist *clist)
#endif #endif
filep = &plist->fl_files[i][j]; filep = &plist->fl_files[i][j];
if (filep->f_inode == NULL || (filep->f_oflags & O_CLOEXEC) != 0) DEBUGASSERT(filep);
if (filep && (filep->f_inode == NULL ||
(filep->f_oflags & O_CLOEXEC) != 0))
{ {
continue; continue;
} }