diff options
| author | Richard Weinberger <richard@nod.at> | 2017-06-07 23:33:35 +0200 |
|---|---|---|
| committer | Richard Weinberger <richard@nod.at> | 2017-07-14 22:49:07 +0200 |
| commit | df71b09145b66e4cf6f7a1ec69d181bf2ccb0efd (patch) | |
| tree | 6d6eb0d66d61ef13a983aa0c1a3b26140fa0a4e9 | |
| parent | 319c10427498620d220931189dcdba7d4244feba (diff) | |
| download | op-kernel-dev-df71b09145b66e4cf6f7a1ec69d181bf2ccb0efd.zip op-kernel-dev-df71b09145b66e4cf6f7a1ec69d181bf2ccb0efd.tar.gz | |
ubifs: Fail commit if TNC is obviously inconsistent
A reference to LEB 0 or with length 0 in the TNC
is never correct and could be caused by a memory corruption.
Don't write such a bad index node to the MTD.
Instead fail the commit which will turn UBIFS into read-only mode.
This is less painful than having the bad reference on the MTD
from where UBFIS has no chance to recover.
Signed-off-by: Richard Weinberger <richard@nod.at>
| -rw-r--r-- | fs/ubifs/tnc_commit.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/ubifs/tnc_commit.c b/fs/ubifs/tnc_commit.c index 51157da..aa31f60 100644 --- a/fs/ubifs/tnc_commit.c +++ b/fs/ubifs/tnc_commit.c @@ -57,6 +57,8 @@ static int make_idx_node(struct ubifs_info *c, struct ubifs_idx_node *idx, ubifs_dump_znode(c, znode); if (zbr->znode) ubifs_dump_znode(c, zbr->znode); + + return -EINVAL; } } ubifs_prepare_node(c, idx, len, 0); @@ -859,6 +861,8 @@ static int write_index(struct ubifs_info *c) ubifs_dump_znode(c, znode); if (zbr->znode) ubifs_dump_znode(c, zbr->znode); + + return -EINVAL; } } len = ubifs_idx_node_sz(c, znode->child_cnt); |
