diff options
author | Douglas Gilbert <dgilbert@interlog.com> | 2018-05-15 20:44:45 +0000 |
---|---|---|
committer | Douglas Gilbert <dgilbert@interlog.com> | 2018-05-15 20:44:45 +0000 |
commit | 856fb91355bddbe7a15a9d11334c354d8ee7b6d3 (patch) | |
tree | 9446f14be2d17b1f0449a466e0762a9a3361ca7b /src/sg_sat_set_features.c | |
parent | 71e56cc16387a76971a04fda4ff9306ec704daf9 (diff) | |
download | sg3_utils-856fb91355bddbe7a15a9d11334c354d8ee7b6d3.tar.gz |
cleanup of error messages around sg_cmds_open_device(); add flock error msg for ddpt
git-svn-id: https://svn.bingwo.ca/repos/sg3_utils/trunk@772 6180dd3e-e324-4e3e-922d-17de1ae2f315
Diffstat (limited to 'src/sg_sat_set_features.c')
-rw-r--r-- | src/sg_sat_set_features.c | 32 |
1 files changed, 22 insertions, 10 deletions
diff --git a/src/sg_sat_set_features.c b/src/sg_sat_set_features.c index 523e38a4..dbcd8353 100644 --- a/src/sg_sat_set_features.c +++ b/src/sg_sat_set_features.c @@ -50,11 +50,12 @@ #define DEF_TIMEOUT 20 -static const char * version_str = "1.16 20180219"; +static const char * version_str = "1.17 20180513"; static struct option long_options[] = { {"count", required_argument, 0, 'c'}, {"ck_cond", no_argument, 0, 'C'}, + {"ck-cond", no_argument, 0, 'C'}, {"extended", no_argument, 0, 'e'}, {"feature", required_argument, 0, 'f'}, {"help", no_argument, 0, 'h'}, @@ -304,7 +305,8 @@ main(int argc, char * argv[]) bool ck_cond = false; bool extend = false; bool rdonly = false; - int sg_fd, c, ret, res; + int c, ret, res; + int sg_fd = -1; int count = 0; int feature = 0; int verbose = 0; @@ -409,19 +411,29 @@ main(int argc, char * argv[]) } if ((sg_fd = sg_cmds_open_device(device_name, rdonly, verbose)) < 0) { - pr2serr("error opening file: %s: %s\n", device_name, - safe_strerror(-sg_fd)); - return SG_LIB_FILE_ERROR; + if (verbose) + pr2serr("error opening file: %s: %s\n", device_name, + safe_strerror(-sg_fd)); + ret = sg_convert_errno(-sg_fd); + goto fini; } ret = do_set_features(sg_fd, feature, count, lba, cdb_len, ck_cond, extend, verbose); - res = sg_cmds_close_device(sg_fd); - if (res < 0) { - pr2serr("close error: %s\n", safe_strerror(-res)); - if (0 == ret) - return SG_LIB_FILE_ERROR; +fini: + if (sg_fd >= 0) { + res = sg_cmds_close_device(sg_fd); + if (res < 0) { + pr2serr("close error: %s\n", safe_strerror(-res)); + if (0 == ret) + ret = sg_convert_errno(-res); + } + } + if (0 == verbose) { + if (! sg_if_can2stderr("sg_sat_set_feature failed: ", ret)) + pr2serr("Some error occurred, try again with '-v' " + "or '-vv' for more information\n"); } return (ret >= 0) ? ret : SG_LIB_CAT_OTHER; } |