Reading the serialno property is forbidden
This read always fails so just remove it to avoid selinux denials.
Bug: 63963344
Test: Ran downloader utility on Angelfish over WIFI
Change-Id: I6ce7bb5d161c9d2c4478b5d0fa23dd64b6d599b3
Merged-In: I6ce7bb5d161c9d2c4478b5d0fa23dd64b6d599b3
(cherry picked from commit 1744da2bed9c6c41188b224da77d0e372d562eed)
diff --git a/wcnss-service/wcnss_service.c b/wcnss-service/wcnss_service.c
index e829db4..2349299 100644
--- a/wcnss-service/wcnss_service.c
+++ b/wcnss-service/wcnss_service.c
@@ -402,10 +402,8 @@
#endif
{
char msg[WCNSS_MAX_CMD_LEN];
- char serial[PROPERTY_VALUE_MAX];
int fd, rc, pos = 0;
struct stat st;
- unsigned int serial_num = 0;
fd = open(WCNSS_CTRL, O_WRONLY);
if (fd < 0) {
@@ -413,25 +411,6 @@
return;
}
- rc = property_get("ro.serialno", serial, "");
- if (rc) {
- serial_num = convert_string_to_hex(serial);
- ALOGE("Serial Number is %x", serial_num);
-
- msg[pos++] = WCNSS_USR_SERIAL_NUM >> BYTE_1;
- msg[pos++] = WCNSS_USR_SERIAL_NUM >> BYTE_0;
- msg[pos++] = serial_num >> BYTE_3;
- msg[pos++] = serial_num >> BYTE_2;
- msg[pos++] = serial_num >> BYTE_1;
- msg[pos++] = serial_num >> BYTE_0;
-
- if (write(fd, msg, pos) < 0) {
- ALOGE("Failed to write to %s : %s", WCNSS_CTRL,
- strerror(errno));
- goto fail;
- }
- }
-
#if defined(WCNSS_QMI) || defined (WCNSS_QMI_OSS)
if (SUCCESS == nv_mac_addr)
{