This is an automated email from the git hooks/post-receive script.
teigland pushed a commit to branch master in repository sanlock.
The following commit(s) were added to refs/heads/master by this push: new 50a1591 sanlock: don't ignore wrong size in read_resource_owners 50a1591 is described below
commit 50a1591165c572b9848b0534956427614b6cabbd Author: David Teigland teigland@redhat.com AuthorDate: Thu Mar 21 13:05:06 2019 -0500
sanlock: don't ignore wrong size in read_resource_owners
If the user specifies the wrong lease size and it triggers an error, then return the error instead of correcting it internally. There is a possibility that we might want a short read to fail in other cases also. --- src/resource.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/src/resource.c b/src/resource.c index 66a5f37..a783106 100644 --- a/src/resource.c +++ b/src/resource.c @@ -188,12 +188,7 @@ int read_resource_owners(struct task *task, struct token *token, /* we could in-line paxos_read_buf here like we do in read_mode_block */ retry: rv = paxos_read_buf(task, token, &lease_buf); - if (rv == -EMSGSIZE) { - /* if an 8M lease was specified, but it's only a 1M lease at - the end of the device, then we'll get an error from a short - read */ - log_token(token, "read_resource_owners read_buf EMSGSIZE"); - } else if (rv < 0) { + if (rv < 0) { log_errot(token, "read_resource_owners read_buf rv %d", rv);
if (lease_buf && (rv != SANLK_AIO_TIMEOUT))
sanlock-devel@lists.fedorahosted.org