diff options
author | Eric Blake <eblake@redhat.com> | 2017-04-29 14:14:17 -0500 |
---|---|---|
committer | Max Reitz <mreitz@redhat.com> | 2017-05-11 14:28:06 +0200 |
commit | 3dc834f8795a46f919d90b1e5369308628858601 (patch) | |
tree | 4073bdb2f11b9e63619ea532ab7f1939e79db361 /migration/fd.c | |
parent | 63188c245013dbe383e8b031e665f813e2452ea5 (diff) |
blkdebug: Simplify override logic
Rather than store into a local variable, then copy to the struct
if the value is valid, then reporting errors otherwise, it is
simpler to just store into the struct and report errors if the
value is invalid. This however requires that the struct store
a 64-bit number, rather than a narrower type. Likewise, setting
a sane errno value in ret prior to the sequence of parsing and
jumping to out: on error makes it easier for the next patch to
add a chain of similar checks.
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-id: 20170429191419.30051-8-eblake@redhat.com
Reviewed-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Max Reitz <mreitz@redhat.com>
Diffstat (limited to 'migration/fd.c')
0 files changed, 0 insertions, 0 deletions