Correct handling of short AIDs will take more than that, but
leaving part of the array uninitialized is wrong in any case.
---
 src/simutil.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/simutil.c b/src/simutil.c
index 5d2aa6a2..e648c918 100644
--- a/src/simutil.c
+++ b/src/simutil.c
@@ -1588,6 +1588,7 @@ GSList *sim_parse_app_template_entries(const unsigned 
char *buffer, int len)
                        goto error;
 
                memcpy(app.aid, aid, app.aid_len);
+               memset(app.aid + app.aid_len, 0xff, 16 - app.aid_len);
 
                app.type = (app.aid[5] << 8) | app.aid[6];
 
-- 
2.25.1
_______________________________________________
ofono mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to