Skip to content

wt-status: fix an invalid memory read, clean up

René Scharfe git 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest5 patches 4 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5t7060: add test for status --branch on a detached HEADt/t7060-wtstatus.sh+14 -00 replies
2/5wt-status: exit early using goto in wt_shortstatus_print_tracking()wt-status.c+5 -80 replies
3/5wt-status: avoid building bogus branch name with detached HEADt/t7060-wtstatus.sh wt-status.c+10 -72 replies
4/5wt-status: don't skip a magical number of characters blindlywt-status.c+1 -12 replies
5/5wt-status: use skip_prefix() to get rid of magic string length constantswt-status.c+15 -210 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/5] wt-status: fix an invalid memory read, clean up

Memory is accessed out-of-bounds when try to show --branch info in a repo with a detached HEAD. Add a test in patch 1 and a fix in patch 3, as well as some cleanups.

Rene Scharfe (5):
t7060: add test for status --branch on a detached HEAD
wt-status: exit early using goto in wt_shortstatus_print_tracking()
wt-status: avoid building bogus branch name with detached HEAD
wt-status: don't skip a magical number of characters blindly
wt-status: use skip_prefix() to get rid of magic string length constants

 t/t7060-wtstatus.sh | 14 ++++++++++++
 wt-status.c         | 64 ++++++++++++++++++++++++-----------------------------
 2 files changed, 43 insertions(+), 35 deletions(-)
Signature · 1 line
2.6.2