diff options
| author | Zhihao Cheng <chengzhihao1@huawei.com> | 2024-11-11 17:07:59 +0800 | 
|---|---|---|
| committer | David Oberhollenzer <david.oberhollenzer@sigma-star.at> | 2024-11-11 10:32:46 +0100 | 
| commit | acedb85997d7b04b4546154d8acc5d916c020630 (patch) | |
| tree | 4a279bb48a709eead122ab37a2c7782015dd6c93 /ubifs-utils/libubifs | |
| parent | 0fbfbf59714b73a0d0c487c454c22d72b2c78e68 (diff) | |
fsck.ubifs: Recover isize
This is the 5/18 step of fsck. Recover isize. There could be following
steps and possible errors:
 Step 1. Traverse size tree, lookup corresponding inode from TNC
  a. corrupted node searched from TNC: skip node for danger mode and
     normal mode with 'yes' answer, other modes will exit.
  b. corrupted index node read from TNC: danger mode with rebuild_fs and
     normal mode with 'yes' answer will turn to rebuild filesystem, other
     modes will exit.
 Step 2. update isize for inode. Keep <inum, isize> in size tree for check
 mode, update inode node in place for other modes.
Signed-off-by: Zhihao Cheng <chengzhihao1@huawei.com>
Signed-off-by: David Oberhollenzer <david.oberhollenzer@sigma-star.at>
Diffstat (limited to 'ubifs-utils/libubifs')
| -rw-r--r-- | ubifs-utils/libubifs/recovery.c | 34 | 
1 files changed, 28 insertions, 6 deletions
| diff --git a/ubifs-utils/libubifs/recovery.c b/ubifs-utils/libubifs/recovery.c index 9115b17..a5133a0 100644 --- a/ubifs-utils/libubifs/recovery.c +++ b/ubifs-utils/libubifs/recovery.c @@ -1272,8 +1272,18 @@ static int fix_size_in_place(struct ubifs_info *c, struct size_entry *e)  	/* Locate the inode node LEB number and offset */  	ino_key_init(c, &key, e->inum);  	err = ubifs_tnc_locate(c, &key, ino, &lnum, &offs); -	if (err) +	if (err) { +		unsigned int reason = get_failure_reason_callback(c); + +		if (reason & FR_DATA_CORRUPTED) { +			test_and_clear_failure_reason_callback(c, FR_DATA_CORRUPTED); +			if (handle_failure_callback(c, FR_H_TNC_DATA_CORRUPTED, NULL)) { +				/* Leave the inode to be deleted by subsequent steps */ +				return 0; +			} +		}  		goto out; +	}  	/*  	 * If the size recorded on the inode node is greater than the size that  	 * was calculated from nodes in the journal then don't change the inode. @@ -1320,10 +1330,10 @@ out:   */  static int inode_fix_size(struct ubifs_info *c, __unused struct size_entry *e)  { -	ubifs_assert(c, 0); - -	// To be implemented -	return -EINVAL; +	/* Don't remove entry, keep it in the size tree. */ +	/* Remove this assertion after supporting authentication. */ +	ubifs_assert(c, c->ro_mount); +	return 0;  }  /** @@ -1353,8 +1363,19 @@ int ubifs_recover_size(struct ubifs_info *c, bool in_place)  			ino_key_init(c, &key, e->inum);  			err = ubifs_tnc_lookup(c, &key, c->sbuf); -			if (err && err != -ENOENT) +			if (err && err != -ENOENT) { +				unsigned int reason; + +				reason = get_failure_reason_callback(c); +				if (reason & FR_DATA_CORRUPTED) { +					test_and_clear_failure_reason_callback(c, FR_DATA_CORRUPTED); +					if (handle_failure_callback(c, FR_H_TNC_DATA_CORRUPTED, NULL)) { +						/* Leave the inode to be deleted by subsequent steps */ +						goto delete_entry; +					} +				}  				return err; +			}  			if (err == -ENOENT) {  				/* Remove data nodes that have no inode */  				dbg_rcvry("removing ino %lu", @@ -1390,6 +1411,7 @@ int ubifs_recover_size(struct ubifs_info *c, bool in_place)  			}  		} +delete_entry:  		rb_erase(&e->rb, &c->size_tree);  		kfree(e);  	} | 
