On Wed, Dec 15, 2021 at 6:11 PM Liuxiangdong [off-list ref] wrote:
Hi, yongji.
In vduse patches serial[1], you said "The support for other device types
can be added after the security issue of corresponding device driver
is clarified or fixed in the future."
What does this "security issue" mean?
[1]https://lore.kernel.org/all/20210831103634.33-1-xieyongji@bytedance.com/ (local)
Do you mean that vduse device is untrusted, so we should check config or
data transferred
from vduse to virtio module? Or something others?
Yes, we need to make sure untrusted devices can not do harm to the
kernel. So we need careful auditing for the device driver before we
add more device types.
Because I found you added some validation in virtio module just like
this patch[2].
[2]https://lore.kernel.org/lkml/20210531135852.113-1-xieyongji@bytedance.com/ (local)
Other efforts are shown below:
https://lwn.net/Articles/865216/
https://lwn.net/Articles/872648/
Thanks,
Yongji