diff --git a/internal/rpc/msg/revoke.go b/internal/rpc/msg/revoke.go
index bc6a7a551..05c38accb 100644
--- a/internal/rpc/msg/revoke.go
+++ b/internal/rpc/msg/revoke.go
@@ -55,11 +55,10 @@ func (m *msgServer) RevokeMsg(ctx context.Context, req *msg.RevokeMsgReq) (*msg.
 				return nil, err
 			}
 			if req.UserID != msgs[0].SendID {
-				roleLevel := members[req.UserID].RoleLevel
 				switch members[req.UserID].RoleLevel {
 				case constant.GroupOwner:
 				case constant.GroupAdmin:
-					if roleLevel != constant.GroupOrdinaryUsers {
+					if members[msgs[0].SendID].RoleLevel != constant.GroupOrdinaryUsers {
 						return nil, errs.ErrNoPermission.Wrap("no permission")
 					}
 				default: