cml/libcw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cwFileSys.h/cpp : Prevent dirEntries from listing '.' and '..' directories during recursion.

Browse Source 
Added dirEntryTest().
This commit is contained in:
kevin 2024-11-30 12:14:29 -05:00
parent ddab771c11
commit a790ab11d3
2 changed files with 66 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

71
cwFileSys.cpp
View File

@ -603,7 +603,7 @@ namespace cw
unsigned passIdx; unsigned passIdx;
} deRecd_t; } deRecd_t;
cw::rc_t _dirGetEntries( deRecd_t* drp, const char* dirStr ) cw::rc_t _dirGetEntries( deRecd_t* drp, const char* dirStr, bool isInRecurseFl=false )
{ {
rc_t rc = kOkRC; rc_t rc = kOkRC;
DIR* dirp = NULL; DIR* dirp = NULL;
@ -660,7 +660,7 @@ namespace cw
{ {
if( strcmp(dp->d_name,".") == 0 ) if( strcmp(dp->d_name,".") == 0 )
{ {
if( cwIsFlag(drp->filterFlags,kCurDirFsFl) == false ) if( isInRecurseFl || cwIsFlag(drp->filterFlags,kCurDirFsFl) == false )
continue; continue;
flags |= kCurDirFsFl; flags |= kCurDirFsFl;
@ -668,7 +668,7 @@ namespace cw
if( strcmp(dp->d_name,"..") == 0 ) if( strcmp(dp->d_name,"..") == 0 )
{ {
if( cwIsFlag(drp->filterFlags,kParentDirFsFl) == false ) if( isInRecurseFl || cwIsFlag(drp->filterFlags,kParentDirFsFl) == false )
continue; continue;
flags |= kParentDirFsFl; flags |= kParentDirFsFl;
@ -703,7 +703,7 @@ namespace cw
flags |= kLinkFsFl; flags |= kLinkFsFl;
if( cwIsFlag(drp->filterFlags,kRecurseLinksFsFl) ) if( cwIsFlag(drp->filterFlags,kRecurseLinksFsFl) )
if((rc = _dirGetEntries(drp,fn)) != kOkRC ) if((rc = _dirGetEntries(drp,fn,true)) != kOkRC )
goto errLabel; goto errLabel;
} }
else else
@ -727,8 +727,8 @@ namespace cw
flags |= kDirFsFl; flags |= kDirFsFl;
if( cwIsFlag(drp->filterFlags,kRecurseFsFl) ) if( cwIsFlag(drp->filterFlags,kRecurseFsFl) && cwIsNotFlag(flags,kCurDirFsFl) && cwIsNotFlag(flags,kParentDirFsFl) )
if((rc = _dirGetEntries(drp,fn)) != kOkRC ) if((rc = _dirGetEntries(drp,fn,true)) != kOkRC )
goto errLabel; goto errLabel;
} }
else else
@ -898,6 +898,65 @@ cw::rc_t cw::filesys::makeDir( const char* dirStr )
return kOkRC; return kOkRC;
} }
cw::rc_t cw::filesys::dirEntryTest( const object_t* cfg )
{
rc_t rc = kOkRC;
unsigned flags = 0;
const char* path = nullptr;
const object_t* attrL = nullptr;
dirEntry_t* de = nullptr;
unsigned deN = 0;
idLabelPair_t map[] = {
{kFileFsFl, "file"},
{kDirFsFl, "dir"},
{kLinkFsFl, "link"},
{kInvisibleFsFl, "hidden"},
{kCurDirFsFl, "cur_dir"},
{kParentDirFsFl, "parent_dir"},
{kFullPathFsFl, "full_path"},
{kRecurseFsFl, "recurse"},
{kRecurseLinksFsFl, "recurse_links" },
{0,nullptr} };
if((rc = cfg->getv("path",path,
"attrL",attrL)) != kOkRC )
{
cwLogError(rc,"Top level parse failed.");
goto errLabel;
}
for(unsigned i=0; i<attrL->child_count(); ++i)
{
const object_t* label = attrL->child_ele(i);
const char* s = nullptr;
unsigned fl = 0;
if( !label->is_string() || label->value(s) != kOkRC || (fl=labelToId(map,s,0))==0 )
{
rc = cwLogError(kSyntaxErrorRC,"The filesys dir entries flag '%s' is not valid.",s==nullptr ? "<empty>" : s);
goto errLabel;
}
flags |= fl;
}
if( flags == 0 )
cwLogWarning("The directory entry inclusion flag is zero (i.e. include nothing).");
if((de = dirEntries( path, flags, &deN )) == nullptr )
{
rc = cwLogError(kOpFailRC,"The directory entry access function failed on '%s' flags:0x%x.",cwStringNullGuard(path),flags);
goto errLabel;
}
for(unsigned i=0; i<deN; ++i)
cwLogInfo("%s",de[i].name);
errLabel:
mem::release(de);
return rc;
}
cw::rc_t cw::filesys::test( const test::test_args_t& args ) cw::rc_t cw::filesys::test( const test::test_args_t& args )
{ {

1
cwFileSys.h
View File

@ -91,6 +91,7 @@ namespace cw
rc_t makeDir( const char* dirStr ); rc_t makeDir( const char* dirStr );
rc_t dirEntryTest( const object_t* cfg );
rc_t test( const test::test_args_t& args ); rc_t test( const test::test_args_t& args );
} }