Merge pull request #100 from rosslagerwall/access-improve
access() improvementslibnfs-4.0.0-vitalif
commit
ab1af6953c
|
@ -1270,6 +1270,36 @@ EXTERN int nfs_access(struct nfs_context *nfs, const char *path, int mode);
|
|||
|
||||
|
||||
|
||||
|
||||
/*
|
||||
* ACCESS2()
|
||||
*/
|
||||
/*
|
||||
* Async access2(<path>)
|
||||
* Function returns
|
||||
* 0 : The operation was initiated. Once the operation finishes, the callback will be invoked.
|
||||
* <0 : An error occured when trying to set up the operation. The callback will not be invoked.
|
||||
*
|
||||
* When the callback is invoked, status indicates the result:
|
||||
* >= 0 : A mask of R_OK, W_OK and X_OK indicating which permissions are
|
||||
* available.
|
||||
* data is NULL
|
||||
* -errno : An error occured.
|
||||
* data is the error string.
|
||||
*/
|
||||
EXTERN int nfs_access2_async(struct nfs_context *nfs, const char *path, nfs_cb cb, void *private_data);
|
||||
/*
|
||||
* Sync access(<path>)
|
||||
* Function returns
|
||||
* >= 0 : A mask of R_OK, W_OK and X_OK indicating which permissions are
|
||||
* available.
|
||||
* -errno : The command failed.
|
||||
*/
|
||||
EXTERN int nfs_access2(struct nfs_context *nfs, const char *path);
|
||||
|
||||
|
||||
|
||||
|
||||
/*
|
||||
* SYMLINK()
|
||||
*/
|
||||
|
|
|
@ -1277,6 +1277,40 @@ int nfs_access(struct nfs_context *nfs, const char *path, int mode)
|
|||
|
||||
|
||||
|
||||
/*
|
||||
* access2()
|
||||
*/
|
||||
static void access2_cb(int status, struct nfs_context *nfs, void *data, void *private_data)
|
||||
{
|
||||
struct sync_cb_data *cb_data = private_data;
|
||||
|
||||
cb_data->is_finished = 1;
|
||||
cb_data->status = status;
|
||||
|
||||
if (status < 0) {
|
||||
nfs_set_error(nfs, "access2 call failed with \"%s\"", (char *)data);
|
||||
return;
|
||||
}
|
||||
}
|
||||
|
||||
int nfs_access2(struct nfs_context *nfs, const char *path)
|
||||
{
|
||||
struct sync_cb_data cb_data;
|
||||
|
||||
cb_data.is_finished = 0;
|
||||
|
||||
if (nfs_access2_async(nfs, path, access2_cb, &cb_data) != 0) {
|
||||
nfs_set_error(nfs, "nfs_access2_async failed");
|
||||
return -1;
|
||||
}
|
||||
|
||||
wait_for_nfs_reply(nfs, &cb_data);
|
||||
|
||||
return cb_data.status;
|
||||
}
|
||||
|
||||
|
||||
|
||||
/*
|
||||
* symlink()
|
||||
*/
|
||||
|
|
108
lib/libnfs.c
108
lib/libnfs.c
|
@ -4314,7 +4314,7 @@ static void nfs_access_cb(struct rpc_context *rpc, int status, void *command_dat
|
|||
ACCESS3res *res;
|
||||
struct nfs_cb_data *data = private_data;
|
||||
struct nfs_context *nfs = data->nfs;
|
||||
unsigned int nfsmode = 0;
|
||||
unsigned int mode = 0;
|
||||
|
||||
assert(rpc->magic == RPC_CONTEXT_MAGIC);
|
||||
|
||||
|
@ -4337,24 +4337,24 @@ static void nfs_access_cb(struct rpc_context *rpc, int status, void *command_dat
|
|||
return;
|
||||
}
|
||||
|
||||
if (data->continue_int & R_OK) {
|
||||
nfsmode |= ACCESS3_READ;
|
||||
if ((data->continue_int & R_OK) && (res->ACCESS3res_u.resok.access & ACCESS3_READ)) {
|
||||
mode |= R_OK;
|
||||
}
|
||||
if (data->continue_int & W_OK) {
|
||||
nfsmode |= ACCESS3_MODIFY;
|
||||
if ((data->continue_int & W_OK) && (res->ACCESS3res_u.resok.access & (ACCESS3_MODIFY | ACCESS3_EXTEND | ACCESS3_DELETE))) {
|
||||
mode |= W_OK;
|
||||
}
|
||||
if (data->continue_int & X_OK) {
|
||||
nfsmode |= ACCESS3_EXECUTE;
|
||||
if ((data->continue_int & X_OK) && (res->ACCESS3res_u.resok.access & (ACCESS3_LOOKUP | ACCESS3_EXECUTE))) {
|
||||
mode |= X_OK;
|
||||
}
|
||||
|
||||
if (res->ACCESS3res_u.resok.access != nfsmode) {
|
||||
if (data->continue_int != mode) {
|
||||
rpc_set_error(nfs->rpc, "NFS: ACCESS denied. Required access %c%c%c. Allowed access %c%c%c",
|
||||
nfsmode&ACCESS3_READ?'r':'-',
|
||||
nfsmode&ACCESS3_MODIFY?'w':'-',
|
||||
nfsmode&ACCESS3_EXECUTE?'x':'-',
|
||||
res->ACCESS3res_u.resok.access&ACCESS3_READ?'r':'-',
|
||||
res->ACCESS3res_u.resok.access&ACCESS3_MODIFY?'w':'-',
|
||||
res->ACCESS3res_u.resok.access&ACCESS3_EXECUTE?'x':'-');
|
||||
data->continue_int&R_OK?'r':'-',
|
||||
data->continue_int&W_OK?'w':'-',
|
||||
data->continue_int&X_OK?'x':'-',
|
||||
mode&R_OK?'r':'-',
|
||||
mode&W_OK?'w':'-',
|
||||
mode&X_OK?'x':'-');
|
||||
data->cb(-EACCES, nfs, rpc_get_error(nfs->rpc), data->private_data);
|
||||
free_nfs_cb_data(data);
|
||||
return;
|
||||
|
@ -4373,10 +4373,10 @@ static int nfs_access_continue_internal(struct nfs_context *nfs, fattr3 *attr _U
|
|||
nfsmode |= ACCESS3_READ;
|
||||
}
|
||||
if (data->continue_int & W_OK) {
|
||||
nfsmode |= ACCESS3_MODIFY;
|
||||
nfsmode |= ACCESS3_MODIFY | ACCESS3_EXTEND | ACCESS3_DELETE;
|
||||
}
|
||||
if (data->continue_int & X_OK) {
|
||||
nfsmode |= ACCESS3_EXECUTE;
|
||||
nfsmode |= ACCESS3_LOOKUP | ACCESS3_EXECUTE;
|
||||
}
|
||||
|
||||
memset(&args, 0, sizeof(ACCESS3args));
|
||||
|
@ -4394,7 +4394,81 @@ static int nfs_access_continue_internal(struct nfs_context *nfs, fattr3 *attr _U
|
|||
|
||||
int nfs_access_async(struct nfs_context *nfs, const char *path, int mode, nfs_cb cb, void *private_data)
|
||||
{
|
||||
if (nfs_lookuppath_async(nfs, path, 0, cb, private_data, nfs_access_continue_internal, NULL, NULL, mode) != 0) {
|
||||
if (nfs_lookuppath_async(nfs, path, 0, cb, private_data, nfs_access_continue_internal, NULL, NULL, mode & (R_OK | W_OK | X_OK)) != 0) {
|
||||
rpc_set_error(nfs->rpc, "Out of memory: failed to start parsing the path components");
|
||||
return -1;
|
||||
}
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
||||
|
||||
/*
|
||||
* Async access2()
|
||||
*/
|
||||
static void nfs_access2_cb(struct rpc_context *rpc, int status, void *command_data, void *private_data)
|
||||
{
|
||||
ACCESS3res *res;
|
||||
struct nfs_cb_data *data = private_data;
|
||||
struct nfs_context *nfs = data->nfs;
|
||||
unsigned int result = 0;
|
||||
|
||||
assert(rpc->magic == RPC_CONTEXT_MAGIC);
|
||||
|
||||
if (status == RPC_STATUS_ERROR) {
|
||||
data->cb(-EFAULT, nfs, command_data, data->private_data);
|
||||
free_nfs_cb_data(data);
|
||||
return;
|
||||
}
|
||||
if (status == RPC_STATUS_CANCEL) {
|
||||
data->cb(-EINTR, nfs, "Command was cancelled", data->private_data);
|
||||
free_nfs_cb_data(data);
|
||||
return;
|
||||
}
|
||||
|
||||
res = command_data;
|
||||
if (res->status != NFS3_OK) {
|
||||
rpc_set_error(nfs->rpc, "NFS: ACCESS of %s failed with %s(%d)", data->saved_path, nfsstat3_to_str(res->status), nfsstat3_to_errno(res->status));
|
||||
data->cb(nfsstat3_to_errno(res->status), nfs, rpc_get_error(nfs->rpc), data->private_data);
|
||||
free_nfs_cb_data(data);
|
||||
return;
|
||||
}
|
||||
|
||||
if (res->ACCESS3res_u.resok.access & ACCESS3_READ) {
|
||||
result |= R_OK;
|
||||
}
|
||||
if (res->ACCESS3res_u.resok.access & (ACCESS3_MODIFY | ACCESS3_EXTEND | ACCESS3_DELETE)) {
|
||||
result |= W_OK;
|
||||
}
|
||||
if (res->ACCESS3res_u.resok.access & (ACCESS3_LOOKUP | ACCESS3_EXECUTE)) {
|
||||
result |= X_OK;
|
||||
}
|
||||
|
||||
data->cb(result, nfs, NULL, data->private_data);
|
||||
free_nfs_cb_data(data);
|
||||
}
|
||||
|
||||
static int nfs_access2_continue_internal(struct nfs_context *nfs, fattr3 *attr _U_, struct nfs_cb_data *data)
|
||||
{
|
||||
ACCESS3args args;
|
||||
|
||||
memset(&args, 0, sizeof(ACCESS3args));
|
||||
args.object = data->fh;
|
||||
args.access = ACCESS3_READ | ACCESS3_LOOKUP | ACCESS3_MODIFY | ACCESS3_EXTEND | ACCESS3_DELETE | ACCESS3_EXECUTE;
|
||||
|
||||
if (rpc_nfs3_access_async(nfs->rpc, nfs_access2_cb, &args, data) != 0) {
|
||||
rpc_set_error(nfs->rpc, "RPC error: Failed to send OPEN ACCESS call for %s", data->path);
|
||||
data->cb(-ENOMEM, nfs, rpc_get_error(nfs->rpc), data->private_data);
|
||||
free_nfs_cb_data(data);
|
||||
return -1;
|
||||
}
|
||||
return 0;
|
||||
}
|
||||
|
||||
int nfs_access2_async(struct nfs_context *nfs, const char *path, nfs_cb cb, void *private_data)
|
||||
{
|
||||
if (nfs_lookuppath_async(nfs, path, 0, cb, private_data, nfs_access2_continue_internal, NULL, NULL, 0) != 0) {
|
||||
rpc_set_error(nfs->rpc, "Out of memory: failed to start parsing the path components");
|
||||
return -1;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue