Package: cogito
Version: 0.18.2-2
Severity: important
Tags: patch

The patch applied to fix bug #402413 introduced a new bug: If $EDITOR is
not set, the _editor() function calls 'editor', which happens to match
the shell function editor(), which in turn calls _editor(), leading to
an infinite recursion. (Segemantation fault of bash, in fact).

A possible fix would be to add a path to the call of 'editor', calling
/usr/bin/editor directly and avoiding the shell function.

Thus, the patch 0002-use-editor-instead-of-vi-if-EDITOR-is-unset.diff
should look like:

diff --git a/cg-Xlib b/cg-Xlib
index c1262bf..e9f1fb9 100755
--- a/cg-Xlib
+++ b/cg-Xlib
@@ -568,7 +568,7 @@ _editor()
        actionname="$1"; shift
        actionkey="$1"; shift
 
-       ${EDITOR:-vi} "$LOGMSG2"
+       ${EDITOR:-/usr/bin/editor} "$LOGMSG2"
        [ -z "$force" ] || return 0
        [ ! "$LOGMSG2" -nt "$LOGMSG" ] || return 0
 


-- System Information:
Debian Release: lenny/sid
  APT prefers unstable
  APT policy: (990, 'unstable'), (500, 'oldstable'), (500, 'testing'), (500, 
'stable'), (101, 'experimental')
Architecture: i386 (i686)

Kernel: Linux 2.6.20.6-vs2.3.0.10-gc39a0604
Locale: LANG=C, [EMAIL PROTECTED] (charmap=ISO-8859-15)
Shell: /bin/sh linked to /bin/bash

Versions of packages cogito depends on:
ii  git-core                     1:1.5.1.1-1 fast, scalable, distributed revisi
ii  patch                        2.5.9-4     Apply a diff file to an original

Versions of packages cogito recommends:
ii  gawk                      1:3.1.5.dfsg-4 GNU awk, a pattern scanning and pr

-- no debconf information


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to