Merge "Fix signedness mismatch and integer underflow"
am: b46a2f7dea
Change-Id: Ie2fcbf4af0ae882d67e7ecdbaf7db56b68eabf36
This commit is contained in:
commit
f0c4361770
1 changed files with 7 additions and 6 deletions
13
cryptfs.cpp
13
cryptfs.cpp
|
@ -2458,24 +2458,25 @@ int cryptfs_changepw(int crypt_type, const char* newpw) {
|
|||
static unsigned int persist_get_max_entries(int encrypted) {
|
||||
struct crypt_mnt_ftr crypt_ftr;
|
||||
unsigned int dsize;
|
||||
unsigned int max_persistent_entries;
|
||||
|
||||
/* If encrypted, use the values from the crypt_ftr, otherwise
|
||||
* use the values for the current spec.
|
||||
*/
|
||||
if (encrypted) {
|
||||
if (get_crypt_ftr_and_key(&crypt_ftr)) {
|
||||
return -1;
|
||||
/* Something is wrong, assume no space for entries */
|
||||
return 0;
|
||||
}
|
||||
dsize = crypt_ftr.persist_data_size;
|
||||
} else {
|
||||
dsize = CRYPT_PERSIST_DATA_SIZE;
|
||||
}
|
||||
|
||||
max_persistent_entries =
|
||||
(dsize - sizeof(struct crypt_persist_data)) / sizeof(struct crypt_persist_entry);
|
||||
|
||||
return max_persistent_entries;
|
||||
if (dsize > sizeof(struct crypt_persist_data)) {
|
||||
return (dsize - sizeof(struct crypt_persist_data)) / sizeof(struct crypt_persist_entry);
|
||||
} else {
|
||||
return 0;
|
||||
}
|
||||
}
|
||||
|
||||
static int persist_get_key(const char* fieldname, char* value) {
|
||||
|
|
Loading…
Reference in a new issue