evemu

Merge lp://staging/~bregma/evemu/lp-884848 into lp://staging/evemu

  1. lp-884848
  2. Merge into trunk
Proposed by Stephen M. Webb
Status: Merged
Merged at revision: 43
Proposed branch: lp://staging/~bregma/evemu/lp-884848
Merge into: lp://staging/evemu
Diff against target: 132 lines (+46/-9)
5 files modified
configure.ac (+3/-0)
data/lp-884848.prop (+27/-0)
src/Makefile.am (+3/-2)
src/evemu.c (+12/-6)
tools/evemu-record.c (+1/-1)
To merge this branch: bzr merge lp://staging/~bregma/evemu/lp-884848
Reviewer Review Type Date Requested Status
Jussi Pakkanen (community) Approve
Review via email: mp+80958@code.staging.launchpad.net

Description of the change

Fixes some buffer overflow problems that were encountered in pratice (Fixes LP: 884848).

To post a comment you must log in.
lp://staging/~bregma/evemu/lp-884848 updated
46. By Stephen M. Webb

Cranked up all warnings and enabled -Werror

47. By Stephen M. Webb

Moved null-termination logic from evemu_write() to evemu_extract(), switched from %a scanf modifier to %m.

Revision history for this message
Stephen M. Webb (bregma) wrote :

Added some additional changes that resulted from enabling compiler warnings.

Revision history for this message
Jussi Pakkanen (jpakkane) wrote :

Looks ok.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches