diff options
| author | Jo-Philipp Wich | 2020-12-23 18:06:12 +0000 |
|---|---|---|
| committer | Jo-Philipp Wich | 2021-01-17 20:14:46 +0000 |
| commit | d3f2041f4363e76fda57dd6b581eb71cf1a0e114 (patch) | |
| tree | be2bf8b4cda53d2baebe7eae1eebee1ec108aabb | |
| parent | ea7f4717f8b210a8fa491d288ff99ef74409dbff (diff) | |
| download | rpcd-d3f2041f4363e76fda57dd6b581eb71cf1a0e114.tar.gz | |
uci: manually clear uci_ptr flags after uci_delete() operations
This is required to avoid potential use-after-free errors through the
uci_set()->uci_delete()->uci_expand_ptr() call chain when passing
zero-length strings as values.
Ref: https://bugs.openwrt.org/index.php?do=details&task_id=3528
Suggested-by: olegio170 <olegios170@gmail.com>
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
| -rw-r--r-- | uci.c | 6 |
1 files changed, 5 insertions, 1 deletions
@@ -831,8 +831,10 @@ rpc_uci_merge_set(struct blob_attr *opt, struct uci_ptr *ptr) if (blobmsg_type(opt) == BLOBMSG_TYPE_ARRAY) { - if (ptr->o) + if (ptr->o) { uci_delete(cursor, ptr); + ptr->flags = 0; + } rv = UBUS_STATUS_INVALID_ARGUMENT; @@ -850,6 +852,7 @@ rpc_uci_merge_set(struct blob_attr *opt, struct uci_ptr *ptr) else if (ptr->o && ptr->o->type == UCI_TYPE_LIST) { uci_delete(cursor, ptr); + ptr->flags = 0; if (!rpc_uci_format_blob(opt, &ptr->value)) return UBUS_STATUS_INVALID_ARGUMENT; @@ -981,6 +984,7 @@ rpc_uci_merge_delete(struct blob_attr *opt, struct uci_ptr *ptr) continue; uci_delete(cursor, ptr); + ptr->flags = 0; rv = 0; } |