[security vulnerability] arbitrary sec kill #54
Description
Recently, our team found an arbitrary sec kill vulnerability in the latest version of the project.
The vulnerability logic is present in the file:https://github.com/newbee-ltd/newbee-mall-plus/blob/main/src/main/java/ltd/newbee/mall/controller/mall/SecKillController.java#102.
The operation ltd.newbee.mall.dao.NewBeeMallSeckillSuccessMapper.getSeckillSuccessByUserIdAndSeckillId(java.lang.Long,java.lang.Long) is not protected by permission checks, which means an attacker can achieve arbitrary sec kill.
To address this vulnerability, we strongly advise that developers implement access control policies that limit API access to admin users or the owner.