diff options
| author | Amitay Isaacs <amitay@gmail.com> | 2013-07-01 16:21:00 +1000 |
|---|---|---|
| committer | Amitay Isaacs <amitay@gmail.com> | 2013-07-02 12:59:08 +1000 |
| commit | cf17247d312a2d5a6cb191c2db7e50bd3bf5bd6f (patch) | |
| tree | 70144cb876d29f7733ddcbcbb524dda24c205b15 | |
| parent | 6fe0089bc0443a1c8ee75f16215a8bd1f9a66cfb (diff) | |
freeze: If priority is invalid here, it's time to abort
ctdb_start_freeze() is called from ctdb_control_freeze() which fixes the
priority if it's 0 and return error if it's invalid. Other callers of
ctdb_start_freeze() are internal to CTDB. So if priority is invalid in
ctdb_start_freeze(), definitely something is seriously wrong.
Signed-off-by: Amitay Isaacs <amitay@gmail.com>
(This used to be ctdb commit 87716e8f504d659515d3dbcf93badbf106873bc8)
| -rw-r--r-- | ctdb/server/ctdb_freeze.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/ctdb/server/ctdb_freeze.c b/ctdb/server/ctdb_freeze.c index 867da6bc92..1acd5c7fa7 100644 --- a/ctdb/server/ctdb_freeze.c +++ b/ctdb/server/ctdb_freeze.c @@ -130,14 +130,9 @@ int ctdb_start_freeze(struct ctdb_context *ctdb, uint32_t priority) { struct ctdb_freeze_handle *h; - if (priority == 0) { - DEBUG(DEBUG_ERR,("Freeze priority 0 requested, remapping to priority 1\n")); - priority = 1; - } - if ((priority < 1) || (priority > NUM_DB_PRIORITIES)) { DEBUG(DEBUG_ERR,(__location__ " Invalid db priority : %u\n", priority)); - return -1; + ctdb_fatal(ctdb, "Internal error"); } if (ctdb->freeze_mode[priority] == CTDB_FREEZE_FROZEN) { |