Browse Source

normalize error handling in rename policy

pull/234/head
Antonio SJ Musumeci 9 years ago
parent
commit
e593927095
  1. 42
      src/rename.cpp
  2. 6
      src/success_fail.hpp

42
src/rename.cpp

@ -28,6 +28,7 @@
#include "fs_path.hpp" #include "fs_path.hpp"
#include "rv.hpp" #include "rv.hpp"
#include "rwlock.hpp" #include "rwlock.hpp"
#include "success_fail.hpp"
#include "ugid.hpp" #include "ugid.hpp"
using std::string; using std::string;
@ -89,7 +90,7 @@ _rename_create_path_core(const vector<const string*> &oldbasepaths,
rv = ::rename(oldfullpath.c_str(),newfullpath.c_str()); rv = ::rename(oldfullpath.c_str(),newfullpath.c_str());
error = calc_error(rv,error,errno); error = calc_error(rv,error,errno);
if(rv == -1)
if(RENAME_FAILED(rv))
tounlink.push_back(oldfullpath); tounlink.push_back(oldfullpath);
} }
else else
@ -114,16 +115,16 @@ _rename_create_path(Policy::Func::Search searchFunc,
vector<const string*> oldbasepaths; vector<const string*> oldbasepaths;
rv = actionFunc(srcmounts,oldfusepath,minfreespace,oldbasepaths); rv = actionFunc(srcmounts,oldfusepath,minfreespace,oldbasepaths);
if(rv == -1)
if(POLICY_FAILED(rv))
return -errno; return -errno;
string newfusedirpath = newfusepath; string newfusedirpath = newfusepath;
fs::path::dirname(newfusedirpath); fs::path::dirname(newfusedirpath);
rv = searchFunc(srcmounts,newfusedirpath.c_str(),minfreespace,newbasepath); rv = searchFunc(srcmounts,newfusedirpath.c_str(),minfreespace,newbasepath);
if(rv == -1)
if(POLICY_FAILED(rv))
return -errno; return -errno;
error = -1;
error = RENAME_FAIL;
for(size_t i = 0, ei = srcmounts.size(); i != ei; i++) for(size_t i = 0, ei = srcmounts.size(); i != ei; i++)
{ {
const string &oldbasepath = srcmounts[i]; const string &oldbasepath = srcmounts[i];
@ -135,7 +136,7 @@ _rename_create_path(Policy::Func::Search searchFunc,
error,toremove); error,toremove);
} }
if(error == 0)
if(RENAME_SUCCEEDED(error))
_remove(toremove); _remove(toremove);
return -error; return -error;
@ -158,25 +159,24 @@ _clonepath_if_would_create(Policy::Func::Search searchFunc,
newfusedirpath = newfusepath; newfusedirpath = newfusepath;
fs::path::dirname(newfusedirpath); fs::path::dirname(newfusedirpath);
rv = createFunc(srcmounts,newfusedirpath.c_str(),minfreespace,newbasepath); rv = createFunc(srcmounts,newfusedirpath.c_str(),minfreespace,newbasepath);
if(rv != -1)
{
if(POLICY_FAILED(rv))
return rv;
if(oldbasepath == *newbasepath[0]) if(oldbasepath == *newbasepath[0])
{ {
rv = searchFunc(srcmounts,newfusedirpath.c_str(),minfreespace,newbasepath); rv = searchFunc(srcmounts,newfusedirpath.c_str(),minfreespace,newbasepath);
if(rv != -1)
if(POLICY_FAILED(rv))
return rv;
{ {
const ugid::SetRootGuard ugidGuard; const ugid::SetRootGuard ugidGuard;
fs::clonepath(*newbasepath[0],oldbasepath,newfusedirpath.c_str()); fs::clonepath(*newbasepath[0],oldbasepath,newfusedirpath.c_str());
} }
}
else
{
rv = -1;
errno = EXDEV;
}
return POLICY_SUCCESS;
} }
return rv;
return POLICY_FAIL_ERRNO(EXDEV);
} }
static static
@ -206,17 +206,17 @@ _rename_preserve_path_core(Policy::Func::Search searchFunc,
fs::path::make(&oldbasepath,oldfusepath,oldfullpath); fs::path::make(&oldbasepath,oldfusepath,oldfullpath);
rv = ::rename(oldfullpath.c_str(),newfullpath.c_str()); rv = ::rename(oldfullpath.c_str(),newfullpath.c_str());
if((rv == -1) && (errno == ENOENT))
if(RENAME_FAILED_WITH(rv,ENOENT))
{ {
rv = _clonepath_if_would_create(searchFunc,createFunc, rv = _clonepath_if_would_create(searchFunc,createFunc,
srcmounts,minfreespace, srcmounts,minfreespace,
oldbasepath,oldfusepath,newfusepath); oldbasepath,oldfusepath,newfusepath);
if(rv != -1)
if(POLICY_SUCCEEDED(rv))
rv = ::rename(oldfullpath.c_str(),newfullpath.c_str()); rv = ::rename(oldfullpath.c_str(),newfullpath.c_str());
} }
error = calc_error(rv,error,errno); error = calc_error(rv,error,errno);
if(rv == -1)
if(RENAME_FAILED(rv))
toremove.push_back(oldfullpath); toremove.push_back(oldfullpath);
} }
else else
@ -241,10 +241,10 @@ _rename_preserve_path(Policy::Func::Search searchFunc,
vector<const string*> oldbasepaths; vector<const string*> oldbasepaths;
rv = actionFunc(srcmounts,oldfusepath,minfreespace,oldbasepaths); rv = actionFunc(srcmounts,oldfusepath,minfreespace,oldbasepaths);
if(rv == -1)
if(POLICY_FAILED(rv))
return -errno; return -errno;
error = -1;
error = RENAME_FAIL;
for(size_t i = 0, ei = srcmounts.size(); i != ei; i++) for(size_t i = 0, ei = srcmounts.size(); i != ei; i++)
{ {
const string &oldbasepath = srcmounts[i]; const string &oldbasepath = srcmounts[i];
@ -256,7 +256,7 @@ _rename_preserve_path(Policy::Func::Search searchFunc,
error,toremove); error,toremove);
} }
if(error == 0)
if(RENAME_SUCCEEDED(error))
_remove(toremove); _remove(toremove);
return -error; return -error;

6
src/success_fail.hpp

@ -36,4 +36,10 @@
#define FSTAT_FAIL -1 #define FSTAT_FAIL -1
#define FSTAT_FAILED(RV) ((RV) == FSTAT_FAIL) #define FSTAT_FAILED(RV) ((RV) == FSTAT_FAIL)
#define RENAME_SUCCESS 0
#define RENAME_SUCCEEDED(RV) ((RV) == RENAME_SUCCESS)
#define RENAME_FAIL -1
#define RENAME_FAILED(RV) ((RV) == RENAME_FAIL)
#define RENAME_FAILED_WITH(RV,ERRNO) (((RV) == RENAME_FAIL) && (errno == ERRNO))
#endif /*__MERGERFS_SUCCESS_FAIL_HPP__ */ #endif /*__MERGERFS_SUCCESS_FAIL_HPP__ */
Loading…
Cancel
Save