Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • L libotr-ng
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 21
    • Issues 21
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • OTRv4
  • libotr-ng
  • Merge requests
  • !206

Merged
Created Aug 09, 2019 by Jurre van Bergen@DrWhaxOwner

Fix snprintf max size error

  • Overview 2
  • Commits 1
  • Changes 1

Created by: lnsndn

Quick and simple fix for #205 (closed)

I have verified that the tests go through on my machine.

@claucece On a related note: there are no checks in this function to verify that hash and human are of the required length. Perhaps this is somehow implied elsewhere throughout the lib? I haven't taken the time to go through everything well enough to understand where they come from and what guarantees exist that the user can't make those mistakes. Or if this is even an important function. This is just my drive-by coding, you of course know better :)

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: github/fork/lnsndn/snprintf