[security vulnerability] Arbitrary Address Update

[GatekeeperBuster]

Recently, our team found a vulnerability causing the Arbitrary Address Update in the latest version of the project.

The vulnerability logic is present in the file: https://github.com/youlaitech/youlai-mall/blob/master/mall-ums/ums-boot/src/main/java/com/youlai/mall/ums/service/impl/UmsAddressServiceImpl.java#73.

The developer failed to check the privilege of the accessor when updating status via com.baomidou.mybatisplus.extension.service.impl.ServiceImpl<OrderMapper,OmsOrder>.getById(), while the request of path /{addressId} is also unauthorized (i.e., https://github.com/youlaitech/youlai-mall/blob/master/mall-ums/ums-boot/src/main/java/com/youlai/mall/ums/controller/app/AddressController.java#L57), which means an attacker can achieve Arbitrary Address Update.

To address this vulnerability, we strongly advise that developers implement access control policies that limit API access to admin users or the owner.