git: sdiff: fix wcwidth recursion
John Marino
marino at crater.dragonflybsd.org
Wed Oct 12 17:07:41 PDT 2011
commit d76863bd1dbf9c1e4c04a977e3a7271a23cdb718
Author: John Marino <draco at marino.st>
Date: Thu Oct 13 00:39:51 2011 +0200
sdiff: fix wcwidth recursion
Apparently trying to diff non-UTF8 documents caused a stack overflow.
The culprit was correctly identified by Yonetani as the redefinition
of wcwidth in the wcwidth.c file. Within that function, it calls itself
recursively forever. I'm not sure how this would work differently
(macro or not) on another system.
My solution was to stop redefining wcwidth and just create a separate
function, one that will call the system wcwidth function as needed.
reported by: Peter Avalos
http://bugs.dragonflybsd.org/issue2134
Summary of changes:
contrib/diffutils/README.DRAGONFLY | 9 +++++----
contrib/diffutils/lib/wcwidth.c | 4 ++--
contrib/diffutils/src/side.c | 3 ++-
gnu/usr.bin/diff/libdiffutils/gnuwidechar.h | 11 +++++++++++
4 files changed, 20 insertions(+), 7 deletions(-)
create mode 100644 gnu/usr.bin/diff/libdiffutils/gnuwidechar.h
http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/d76863bd1dbf9c1e4c04a977e3a7271a23cdb718
--
DragonFly BSD source repository
More information about the Commits
mailing list