Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • L libXRes
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1
    • Issues 1
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar

Due to an influx of spam, we have had to impose restrictions on new accounts. Please see this wiki page for instructions on how to get full permissions. Sorry for the inconvenience.

  • xorgxorg
  • lib
  • libXRes
  • Merge requests
  • !1

XRes: Use _XRead instead of _XRead32 for reading pid

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Ray Strode requested to merge halfline/libxres:xres-fix into master Oct 21, 2020
  • Overview 6
  • Commits 1
  • Pipelines 0
  • Changes 1

_XRead32 requires the output variable to be a long, but ReadClientValues feeds it client->value which is a 32-bit allocation.

This commit changes the code to use _XRead instead, which properly handles 32-bit output variables.

#3 (closed)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: xres-fix