Fix 'missing break' issues found by Coverity version 6.5.1:
Missing break in switch (MISSING_BREAK)
This case is not terminated by a 'break' statement.
Signed-off-by: Sebastien Buisson <sebastien.buisson@bull.net>
Change-Id: Ib96784433db5075bb6fb916024c1c217850f7eb6
Reviewed-on: http://review.whamcloud.com/7164
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: James Nunez <james.a.nunez@intel.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
break;
case CLS_HELD:
CL_LOCK_DEBUG(D_ERROR, env, parent, "Delete CLS_HELD lock\n");
+ /* falling through */
default:
CERROR("Impossible state: %d\n", parent->cll_state);
LBUG();
}
break;
case 'Y':
- if (fdatasync(fd) == -1) {
- save_errno = errno;
- perror("fdatasync");
- exit(save_errno);
- }
+ if (fdatasync(fd) == -1) {
+ save_errno = errno;
+ perror("fdatasync");
+ exit(save_errno);
+ }
+ break;
case 'z':
len = atoi(commands+1);
if (lseek(fd, len, SEEK_SET) == -1) {
{0, 0, 0, 0}
};
- optind = 0;
+ optind = 0;
while ((c = getopt_long(argc, argv, "hilp:", long_opts, NULL)) != -1) {
- switch (c) {
- case 'i':
- ishow = 1;
- break;
- case 'h':
- cooked = 1;
- break;
- case 'l':
- lazy = 1;
- case 'p':
- pool_name = optarg;
- break;
- default:
- return CMD_HELP;
- }
- }
+ switch (c) {
+ case 'i':
+ ishow = 1;
+ break;
+ case 'h':
+ cooked = 1;
+ break;
+ case 'l':
+ lazy = 1;
+ break;
+ case 'p':
+ pool_name = optarg;
+ break;
+ default:
+ return CMD_HELP;
+ }
+ }
if (optind < argc && !realpath(argv[optind], path)) {
rc = -errno;
fprintf(stderr, "error: invalid path '%s': %s\n",