selinux: Add boundary check in put_entry()
authorXiu Jianfeng <xiujianfeng@huawei.com>
Tue, 14 Jun 2022 02:14:49 +0000 (10:14 +0800)
committerPaul Moore <paul@paul-moore.com>
Wed, 15 Jun 2022 01:52:37 +0000 (21:52 -0400)
Just like next_entry(), boundary check is necessary to prevent memory
out-of-bound access.

Signed-off-by: Xiu Jianfeng <xiujianfeng@huawei.com>
Signed-off-by: Paul Moore <paul@paul-moore.com>
security/selinux/ss/policydb.h

index c24d4e1063ea006bd809b16b83e0e718b9e56c66..ffc4e7bad20542ef9453e422c4f54b4b49e8d5db 100644 (file)
@@ -370,6 +370,8 @@ static inline int put_entry(const void *buf, size_t bytes, int num, struct polic
 {
        size_t len = bytes * num;
 
+       if (len > fp->len)
+               return -EINVAL;
        memcpy(fp->data, buf, len);
        fp->data += len;
        fp->len -= len;