diff options
author | Colin Walters <walters@verbum.org> | 2014-02-20 18:25:56 -0500 |
---|---|---|
committer | Colin Walters <walters@verbum.org> | 2014-02-20 18:25:56 -0500 |
commit | f2e0162846cbb84abf6fd9a38fabdfd588f4eb2f (patch) | |
tree | 8d9ca05c4cc01f535e8441fc48c452bc6045a602 /src/ostree/ot-admin-builtin-upgrade.c | |
parent | 9fc9008af8374e7c983a26c18f200e6c7bbf8cbb (diff) | |
download | ostree-f2e0162846cbb84abf6fd9a38fabdfd588f4eb2f.tar.gz |
upgrade: Refuse chronologically older commits unless --allow-downgrade
We don't want to allow MITM attackers to intercept upgrade requests
and provide clients with older OS versions vulnerable to security
flaws.
Only "ostree admin upgrade" gets this behavior for now - whether we
want to do it for "ostree admin switch" is another question.
Diffstat (limited to 'src/ostree/ot-admin-builtin-upgrade.c')
-rw-r--r-- | src/ostree/ot-admin-builtin-upgrade.c | 40 |
1 files changed, 40 insertions, 0 deletions
diff --git a/src/ostree/ot-admin-builtin-upgrade.c b/src/ostree/ot-admin-builtin-upgrade.c index 1b86cddc..9fcae678 100644 --- a/src/ostree/ot-admin-builtin-upgrade.c +++ b/src/ostree/ot-admin-builtin-upgrade.c @@ -34,11 +34,13 @@ #include <glib/gi18n.h> static gboolean opt_reboot; +static gboolean opt_allow_downgrade; static char *opt_osname; static GOptionEntry options[] = { { "os", 0, 0, G_OPTION_ARG_STRING, &opt_osname, "Specify operating system root to use", NULL }, { "reboot", 'r', 0, G_OPTION_ARG_NONE, &opt_reboot, "Reboot after a successful upgrade", NULL }, + { "allow-downgrade", 0, 0, G_OPTION_ARG_NONE, &opt_allow_downgrade, "Permit deployment of chronologically older trees", NULL }, { NULL } }; @@ -109,6 +111,44 @@ ot_admin_builtin_upgrade (int argc, char **argv, OstreeSysroot *sysroot, GCancel else { gs_unref_object GFile *real_sysroot = g_file_new_for_path ("/"); + + if (!opt_allow_downgrade) + { + const char *old_revision = ostree_deployment_get_csum (merge_deployment); + gs_unref_variant GVariant *old_commit = NULL; + gs_unref_variant GVariant *new_commit = NULL; + + if (!ostree_repo_load_variant (repo, OSTREE_OBJECT_TYPE_COMMIT, + old_revision, + &old_commit, + error)) + goto out; + + if (!ostree_repo_load_variant (repo, OSTREE_OBJECT_TYPE_COMMIT, + new_revision, &new_commit, + error)) + goto out; + + if (ostree_commit_get_timestamp (old_commit) > ostree_commit_get_timestamp (new_commit)) + { + GDateTime *old_ts = g_date_time_new_from_unix_utc (ostree_commit_get_timestamp (old_commit)); + GDateTime *new_ts = g_date_time_new_from_unix_utc (ostree_commit_get_timestamp (new_commit)); + gs_free char *old_ts_str = NULL; + gs_free char *new_ts_str = NULL; + + g_assert (old_ts); + g_assert (new_ts); + old_ts_str = g_date_time_format (old_ts, "%c"); + new_ts_str = g_date_time_format (new_ts, "%c"); + g_date_time_unref (old_ts); + g_date_time_unref (new_ts); + + g_set_error (error, G_IO_ERROR, G_IO_ERROR_FAILED, + "Upgrade target revision '%s' with timestamp '%s' is chronologically older than current revision '%s' with timestamp '%s'; use --allow-downgrade to permit", + new_revision, new_ts_str, old_revision, old_ts_str); + goto out; + } + } /* Here we perform cleanup of any leftover data from previous * partial failures. This avoids having to call gs_shutil_rm_rf() |