diff --git a/packages/hydrooj/package.json b/packages/hydrooj/package.json
index 71f6d2016..cc1cbebd6 100644
--- a/packages/hydrooj/package.json
+++ b/packages/hydrooj/package.json
@@ -1,6 +1,6 @@
 {
     "name": "hydrooj",
-    "version": "2.24.7",
+    "version": "2.24.8",
     "bin": "bin/hydrooj.js",
     "main": "dist/loader.js",
     "typings": "dist/loader.d.ts",
diff --git a/packages/hydrooj/src/model/domain.ts b/packages/hydrooj/src/model/domain.ts
index 5811682d9..c53886330 100644
--- a/packages/hydrooj/src/model/domain.ts
+++ b/packages/hydrooj/src/model/domain.ts
@@ -118,10 +118,9 @@ class DomainModel {
             deleteUserCache(udoc);
             return res;
         }
-        const filter = { domainId, uid: { $in: uid } };
-        const affected = await UserModel.getMulti(filter).project({ mail: 1, uname: 1 }).toArray();
+        const affected = await UserModel.getMulti({ _id: { $in: uid } }).project({ mail: 1, uname: 1 }).toArray();
         affected.forEach((udoc) => deleteUserCache(udoc));
-        return await collUser.updateMany(filter, { $set: { role } }, { upsert: true });
+        return await collUser.updateMany({ domainId, uid: { $in: uid } }, { $set: { role } }, { upsert: true });
     }
 
     static async getRoles(domainId: string, count?: boolean): Promise<any[]>;