mirror of
				https://git.code.sf.net/p/linux-ima/ima-evm-utils
				synced 2025-10-30 20:52:15 +01:00 
			
		
		
		
	Revert "Reset 'errno' after failure to open or access a file"
This reverts commit acb19d1894, based on
the mailing list discussion and will be fixed in the next commit.
Reviewed-by: Stefan Berger <stefanb@linux.ibm.com>
Reviewed-by: Petr Vorel <pvorel@suse.cz>
Link: https://lore.kernel.org/linux-integrity/20220915153659.dtykhzitxdrlpasq@altlinux.org/
Signed-off-by: Mimi Zohar <zohar@linux.ibm.com>
			
			
This commit is contained in:
		
							
								
								
									
										18
									
								
								src/evmctl.c
									
									
									
									
									
								
							
							
						
						
									
										18
									
								
								src/evmctl.c
									
									
									
									
									
								
							| @@ -182,7 +182,6 @@ static int bin2file(const char *file, const char *ext, const unsigned char *data | ||||
| 	fp = fopen(name, "w"); | ||||
| 	if (!fp) { | ||||
| 		log_err("Failed to open: %s\n", name); | ||||
| 		errno = 0; | ||||
| 		return -1; | ||||
| 	} | ||||
| 	err = fwrite(data, len, 1, fp); | ||||
| @@ -208,7 +207,6 @@ static unsigned char *file2bin(const char *file, const char *ext, int *size) | ||||
| 	fp = fopen(name, "r"); | ||||
| 	if (!fp) { | ||||
| 		log_err("Failed to open: %s\n", name); | ||||
| 		errno = 0; | ||||
| 		return NULL; | ||||
| 	} | ||||
| 	if (fstat(fileno(fp), &stats) == -1) { | ||||
| @@ -315,10 +313,8 @@ static int get_uuid(struct stat *st, char *uuid) | ||||
| 	sprintf(path, "blkid -s UUID -o value /dev/block/%u:%u", major, minor); | ||||
|  | ||||
| 	fp = popen(path, "r"); | ||||
| 	if (!fp) { | ||||
| 		errno = 0; | ||||
| 	if (!fp) | ||||
| 		goto err; | ||||
| 	} | ||||
|  | ||||
| 	len = fread(_uuid, 1, sizeof(_uuid), fp); | ||||
| 	pclose(fp); | ||||
| @@ -375,7 +371,6 @@ static int calc_evm_hash(const char *file, unsigned char *hash) | ||||
|  | ||||
| 			if (fd < 0) { | ||||
| 				log_err("Failed to open: %s\n", file); | ||||
| 				errno = 0; | ||||
| 				return -1; | ||||
| 			} | ||||
| 			if (ioctl(fd, FS_IOC_GETVERSION, &generation)) { | ||||
| @@ -1201,7 +1196,6 @@ static int calc_evm_hmac(const char *file, const char *keyfile, unsigned char *h | ||||
|  | ||||
| 		if (fd < 0) { | ||||
| 			log_err("Failed to open %s\n", file); | ||||
| 			errno = 0; | ||||
| 			goto out; | ||||
| 		} | ||||
| 		if (ioctl(fd, FS_IOC_GETVERSION, &generation)) { | ||||
| @@ -1392,7 +1386,6 @@ static int ima_fix(const char *path) | ||||
| 	fd = open(path, O_RDONLY); | ||||
| 	if (fd < 0) { | ||||
| 		log_errno("Failed to open file: %s", path); | ||||
| 		errno = 0; | ||||
| 		return -1; | ||||
| 	} | ||||
|  | ||||
| @@ -1916,10 +1909,8 @@ static int read_sysfs_pcrs(int num_banks, struct tpm_bank_info *tpm_banks) | ||||
| 	int i, result; | ||||
|  | ||||
| 	fp = fopen(pcrs, "r"); | ||||
| 	if (!fp) { | ||||
| 	if (!fp) | ||||
| 		fp = fopen(misc_pcrs, "r"); | ||||
| 		errno = 0; | ||||
| 	} | ||||
| 	if (!fp) | ||||
| 		return -1; | ||||
|  | ||||
| @@ -1982,7 +1973,6 @@ static int read_file_pcrs(int num_banks, struct tpm_bank_info *tpm_banks) | ||||
| 		fp = fopen(path, "r"); | ||||
| 		if (!fp) { | ||||
| 			log_err("Could not open '%s'\n", path); | ||||
| 			errno = 0; | ||||
| 			return -1; | ||||
| 		} | ||||
|  | ||||
| @@ -2075,7 +2065,6 @@ static int ima_measurement(const char *file) | ||||
| 	fp = fopen(file, "rb"); | ||||
| 	if (!fp) { | ||||
| 		log_err("Failed to open measurement file: %s\n", file); | ||||
| 		errno = 0; | ||||
| 		return -1; | ||||
| 	} | ||||
|  | ||||
| @@ -2321,7 +2310,6 @@ static int read_binary_bios_measurements(char *file, struct tpm_bank_info *bank) | ||||
| 	fp = fopen(file, "r"); | ||||
| 	if (!fp) { | ||||
| 		log_errno("Failed to open TPM 1.2 event log.\n"); | ||||
| 		errno = 0; | ||||
| 		return 1; | ||||
| 	} | ||||
|  | ||||
| @@ -2758,8 +2746,6 @@ int main(int argc, char *argv[]) | ||||
| 	unsigned long keyid; | ||||
| 	char *eptr; | ||||
|  | ||||
| 	errno = 0;	/* initialize errno */ | ||||
|  | ||||
| #if !(OPENSSL_VERSION_NUMBER < 0x10100000) | ||||
| 	OPENSSL_init_crypto( | ||||
| #ifndef DISABLE_OPENSSL_CONF | ||||
|   | ||||
		Reference in New Issue
	
	Block a user
	 Mimi Zohar
					Mimi Zohar