From f55c23519e3aeffb606f9f68a2c4fa435de84e72 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Rafa=C5=82=20Mi=C5=82ecki?= Date: Fri, 30 Aug 2019 17:46:07 +0200 Subject: [PATCH] system: reject sysupgrade of broken firmware images MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit This uses recently added "validate_firmware_image" to validate passed firmware. If it happens to be invalid and marked as impossible to force then sysupgrade simply exits with an error. This change is needed to avoid bricking devices with some totally broken images. Signed-off-by: Rafał Miłecki --- system.c | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+) diff --git a/system.c b/system.c index 35d5a23..7d7073d 100644 --- a/system.c +++ b/system.c @@ -507,7 +507,18 @@ static int sysupgrade(struct ubus_context *ctx, struct ubus_object *obj, struct ubus_request_data *req, const char *method, struct blob_attr *msg) { + enum { + VALIDATION_VALID, + VALIDATION_FORCEABLE, + __VALIDATION_MAX + }; + static const struct blobmsg_policy validation_policy[__VALIDATION_MAX] = { + [VALIDATION_VALID] = { .name = "valid", .type = BLOBMSG_TYPE_BOOL }, + [VALIDATION_FORCEABLE] = { .name = "forceable", .type = BLOBMSG_TYPE_BOOL }, + }; + struct blob_attr *validation[__VALIDATION_MAX]; struct blob_attr *tb[__SYSUPGRADE_MAX]; + bool valid, forceable; if (!msg) return UBUS_STATUS_INVALID_ARGUMENT; @@ -516,6 +527,21 @@ static int sysupgrade(struct ubus_context *ctx, struct ubus_object *obj, if (!tb[SYSUPGRADE_PATH] || !tb[SYSUPGRADE_PREFIX]) return UBUS_STATUS_INVALID_ARGUMENT; + if (validate_firmware_image_call(blobmsg_get_string(tb[SYSUPGRADE_PATH]))) + return UBUS_STATUS_UNKNOWN_ERROR; + + blobmsg_parse(validation_policy, __VALIDATION_MAX, validation, blob_data(b.head), blob_len(b.head)); + + valid = validation[VALIDATION_VALID] && blobmsg_get_bool(validation[VALIDATION_VALID]); + forceable = validation[VALIDATION_FORCEABLE] && blobmsg_get_bool(validation[VALIDATION_FORCEABLE]); + + if (!valid) { + if (!forceable) { + fprintf(stderr, "Firmware image is broken and cannot be installed\n"); + return UBUS_STATUS_NOT_SUPPORTED; + } + } + sysupgrade_exec_upgraded(blobmsg_get_string(tb[SYSUPGRADE_PREFIX]), blobmsg_get_string(tb[SYSUPGRADE_PATH]), tb[SYSUPGRADE_COMMAND] ? blobmsg_get_string(tb[SYSUPGRADE_COMMAND]) : NULL, -- 2.25.1