GNU bug report logs - #32609
26.1; mail-strip-quoted-names:205

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 32609 in the body.
You can then email your comments to 32609 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Johannes Bruegmann <johannes.bruegmann <at> web.de>
To: bug-gnu-emacs <at> gnu.org
Subject: 26.1; mail-strip-quoted-names:205
Date: Sat, 01 Sep 2018 23:15:26 +0200

I   tried  to   reply  to   an  email   by  pressing   r,  R,   and  M-x
gnus-article-reply-with-original.   None  of  them  worked.   Minibuffer
displayed  "Mark  Set" but  nothing  happened.  I enabled  Debugging  on
C-g. The *Backtrace*-buffer showed
     string-match("\\`[ \t\n]*" "")
     mail-strip-quoted-names("")
     mail-dont-reply-to("foo <at> bar.com")

To me it  looked like that the signature for  string-match and match-end
changed   (or   have   been   overloaded?).  Anyway,   when   i   change
mail-utils.el:205 into
     ;; strip surrounding whitespace
     (setq address (substring address
                              (string-match "\\`[ \t\n]*" address)
                              (string-match "[ \t\n]*\\'" address
                              (match-end 0))))

things work for me.



In GNU Emacs 26.1 (build 1, x86_64-w64-mingw32)
 of 2018-05-30 built on CIRROCUMULUS
Repository revision: 07f8f9bc5a51f5aa94eb099f3e15fbe0c20ea1ea
Windowing system distributor 'Microsoft Corp.', version 10.0.17134
Recent messages:
command-execute: Command attempted to use minibuffer while in minibuffer
Quit [3 times]
Making completion list...
Quit
completing-read-default: Command attempted to use minibuffer while in minibuffer
Quit
command-execute: Command attempted to use minibuffer while in minibuffer
Quit
Making completion list...
Type C-x 1 to delete the help window.
user-error: No cross-reference here
Configured using:
 'configure           --without-dbus           --host=x86_64-w64-mingw32
 --without-compress-install 'CFLAGS=-O2 -static -g3''

Configured features:
XPM  JPEG  TIFF GIF  PNG  RSVG  SOUND  NOTIFY  ACL GNUTLS  LIBXML2  ZLIB
TOOLKIT_SCROLL_BARS THREADS LCMS2

Important settings:
  value of $LANG: DEU
  locale-coding-system: cp1252

Major mode: Info

Minor modes in effect:
  diff-auto-refine-mode: t
  debbugs-browse-mode: t
  shell-dirtrack-mode: t
  global-flycheck-mode: t
  display-time-mode: t
  csv-field-index-mode: t
  tooltip-mode: t
  global-eldoc-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  buffer-read-only: t
  column-number-mode: t
  line-number-mode: t
  transient-mark-mode: t

Load-path shadows:
c:/Users/johan/AppData/Roaming/.emacs.d/elpa/markdown-mode-20180731.1830/markdown-mode hides ~/.emacs.d/elpa/markdown-mode-2.0/markdown-mode
c:/Users/johan/AppData/Roaming/.emacs.d/elpa/markdown-mode-20180731.1830/markdown-mode-pkg hides ~/.emacs.d/elpa/markdown-mode-2.0/markdown-mode-pkg
c:/Users/johan/AppData/Roaming/.emacs.d/elpa/markdown-mode-20180731.1830/markdown-mode-autoloads hides ~/.emacs.d/elpa/markdown-mode-2.0/markdown-mode-autoloads

Features:
(shadow  pulse emacsbug  diff-mode  org-rmail  org-mhe org-irc  org-info
org-gnus org-docview  org-bibtex bibtex org-bbdb org-w3m  nndoc gnus-dup
debbugs-browse org-element avl-tree generator org org-macro org-footnote
org-pcomplete org-list org-faces  org-entities org-version ob-emacs-lisp
ob  ob-tangle org-src  ob-ref ob-lob  ob-table ob-keys  ob-exp ob-comint
ob-core  ob-eval  org-compat  org-macs  org-loaddefs  cal-menu  calendar
cal-loaddefs  debbugs-gnu add-log  debbugs soap-client  warnings rng-xsd
rng-dt rng-util  xsd-regexp xml  autoload tar-mode pp  cus-edit misearch
multi-isearch   mm-archive  url-http   url-gw  url-cache   url-auth  url
url-proxy  url-privacy  url-expand  url-methods  url-history  url-cookie
url-domsuf  url-util   browse-url  finder  lisp-mnt   rfc2104  mailalias
eieio-opt speedbar  sb-image ezimage dframe help-fns  radix-tree apropos
bbdb-message  nnir smiley  mail-extr  gnus-async  gnus-bcklg qp  gnus-ml
cursor-sensor utf-7 nndraft nnmh nnmaildir network-stream nsm gnus-agent
gnus-srvr   nnvirtual   nntp   gnus-cache   smooth-scrolling   elec-pair
fsharp-mode  fsharp-mode-indent-smie   smie  inf-fsharp-mode  fsharp-doc
flycheck-fsharp   fsharp-mode-completion  fsharp-mode-util   etags  xref
project  merlin-company  company  s  tramp  tramp-compat  tramp-loaddefs
trampver     ucs-normalize    shell     pcomplete    flycheck-clangcheck
flycheck-cstyle   flycheck-ocaml  pcase   merlin-cap  merlin   let-alist
flycheck cl-extra  json map  find-func help-mode subr-x  dash finder-inf
info advice  package url-handlers time cus-start  cus-load markdown-mode
color url-parse url-vars thingatpt  noutline outline easy-mmode diatheke
smtpmail  auth-source  cl-seq  eieio eieio-core  cl-macs  eieio-loaddefs
sendmail starttls  moy-bbdb bbdb-gnus  bbdb-mua bbdb-pgp  spam spam-stat
bbdb-com crm  bbdb bbdb-site gnus-uu yenc  gnus-msg gnus-cite gnus-score
score-mode gnus-art  mm-uu mml2015 mm-view mml-smime  smime dig gnus-sum
gnus-group gnus-undo gnus-start gnus-cloud nnimap nnmail mail-source tls
gnutls utf7  netrc nnoo gnus-spec  gnus-int gnus-range message  rmc puny
seq byte-opt bytecomp byte-compile  cconv format-spec rfc822 mml mml-sec
password-cache  epa  epg   epg-config  mailabbrev  gmm-utils  mailheader
gnus-win  gnus   nnheader  gnus-util  rmail   rmail-loaddefs  mail-utils
mm-decode mm-bodies mm-encode w3m  parse-time mail-parse rfc2231 rfc2047
rfc2045 mm-util  ietf-drums mail-prsvr mailcap doc-view  jka-compr dired
dired-loaddefs image-mode timezone  w3m-hist w3m-fb bookmark-w3m w3m-ems
w3m-ccl ccl w3m-favicon w3m-image w3m-proc w3m-util color-theme wid-edit
cl  gv   smooth-scroll  maude-mode  rx  imenu   derived  compile  comint
ansi-color ring csv-mode sort x-dict easymenu edmacro kmacro cl-loaddefs
cl-lib time-date  mule-util tooltip  eldoc electric  uniquify ediff-hook
vc-hooks lisp-float-type mwheel  dos-w32 ls-lisp disp-table term/w32-win
w32-win w32-vars  term/common-win tool-bar dnd fontset  image regexp-opt
fringe tabulated-list replace  newcomment text-mode elisp-mode lisp-mode
prog-mode  register  page  menu-bar  rfn-eshadow  isearch  timer  select
scroll-bar   mouse   jit-lock   font-lock  syntax   facemenu   font-core
term/tty-colors  frame  cl-generic  cham georgian  utf-8-lang  misc-lang
vietnamese tibetan  thai tai-viet  lao korean japanese  eucjp-ms cp51932
hebrew  greek romanian  slovak czech  european ethiopic  indian cyrillic
chinese composite charscript  charprop case-table epa-hook jka-cmpr-hook
help  simple abbrev  obarray  minibuffer  cl-preloaded nadvice  loaddefs
button faces  cus-face macroexp  files text-properties overlay  sha1 md5
base64 format env code-pages mule custom widget hashtable-print-readable
backquote w32notify w32 lcms2 multi-tty make-network-process emacs)

Memory information:
((conses 16 1886794 307219)
 (symbols 56 53561 47)
 (miscs 48 633 853)
 (strings 32 291067 74051)
 (string-bytes 1 7355798)
 (vectors 16 103222)
 (vector-slots 8 2268205 69940)
 (floats 8 3332 1317)
 (intervals 56 185927 1424)
 (buffers 992 57))

Message #8 received at 32609 <at> debbugs.gnu.org (full text, mbox):

From: Noam Postavsky <npostavs <at> gmail.com>
To: Johannes Bruegmann <johannes.bruegmann <at> web.de>
Cc: 32609 <at> debbugs.gnu.org
Subject: Re: bug#32609: 26.1; mail-strip-quoted-names:205
Date: Sun, 02 Sep 2018 07:50:33 -0400

tags 32609 + moreinfo
quit

Johannes Bruegmann <johannes.bruegmann <at> web.de> writes:

> I   tried  to   reply  to   an  email   by  pressing   r,  R,   and  M-x
> gnus-article-reply-with-original.   None  of  them  worked.   Minibuffer
> displayed  "Mark  Set" but  nothing  happened.  I enabled  Debugging  on
> C-g. The *Backtrace*-buffer showed
>      string-match("\\`[ \t\n]*" "")
>      mail-strip-quoted-names("")
>      mail-dont-reply-to("foo <at> bar.com")

If I evaluate (mail-dont-reply-to "foo <at> bar.com") it doesn't hang.  I
guess "foo <at> bar.com" is a placeholder, but you've dropped the relevant
special characters that cause the problem.  Can you give a value that
reproduces the hang?  E.g., just change all letters in the real email to
x, but leave other characters untouched.

> To me it  looked like that the signature for  string-match and match-end
> changed   (or   have   been   overloaded?).  Anyway,   when   i   change

Not sure what you mean about signature change.

> mail-utils.el:205 into
>      ;; strip surrounding whitespace
>      (setq address (substring address
>                               (string-match "\\`[ \t\n]*" address)
>                               (string-match "[ \t\n]*\\'" address
>                               (match-end 0))))
>
> things work for me.

I don't think this change is correct, it doesn't remove the leading whitespace
like before.

Message #13 received at 32609 <at> debbugs.gnu.org (full text, mbox):

From: Noam Postavsky <npostavs <at> gmail.com>
To: 32609 <at> debbugs.gnu.org
Cc: Dr. Johannes Brügmann <dr.johannes.bruegmann <at> gmail.com>
Subject: Re: bug#32609: 26.1; mail-strip-quoted-names:205
Date: Sun, 02 Sep 2018 13:59:39 -0400

[Message part 1 (text/plain, inline)]

[forwarding to list, since sending to newsgroup doesn't get archived in
bug thread]

[Message part 2 (message/rfc822, inline)]

From: Dr. Johannes Brügmann
 <dr.johannes.bruegmann <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: Re: bug#32609: 26.1; mail-strip-quoted-names:205
Date: Sun, 02 Sep 2018 14:56:35 +0200

Noam Postavsky <npostavs <at> gmail.com> writes:

> tags 32609 + moreinfo
> quit
>
> Johannes Bruegmann <johannes.bruegmann <at> web.de> writes:
>
>> I   tried  to   reply  to   an  email   by  pressing   r,  R,   and  M-x
>> gnus-article-reply-with-original.   None  of  them  worked.   Minibuffer
>> displayed  "Mark  Set" but  nothing  happened.  I enabled  Debugging  on
>> C-g. The *Backtrace*-buffer showed
>>      string-match("\\`[ \t\n]*" "")
>>      mail-strip-quoted-names("")
>>      mail-dont-reply-to("foo <at> bar.com")
>
> If I evaluate (mail-dont-reply-to "foo <at> bar.com") it doesn't hang.  I
> guess "foo <at> bar.com" is a placeholder, but you've dropped the relevant
> special characters that cause the problem.  Can you give a value that
> reproduces the hang?  E.g., just change all letters in the real email to
> x, but leave other characters untouched.

The  mail address  i  wanted  to reply  to  didn't  contain any  special
characters.  Neither did  the call  to mail-dont-reply-to.  I understand
that special characters could cause such a behaviour.

As part of following your question, I've put the original file back into
its place, and  then i recompiled using  byte-compile-file and restarted
emacs.   Now the  symtom is  no  longer reproducible.  Neither with  the
originating email, nor with any others that i tested.

>> To me it  looked like that the signature for  string-match and match-end
>> changed   (or   have   been   overloaded?).  Anyway,   when   i   change
>
> Not sure what you mean about signature change.

I  didn't   look  very   carefully  at   the  function   description  of
string-match.   I was  wondering why  someone would  not use  the return
value  of string-match.  Now  i understand  that  string-match holds  an
internal state and match-end can be used to get beyond the first match.

>> mail-utils.el:205 into
>>      ;; strip surrounding whitespace
>>      (setq address (substring address
>>                               (string-match "\\`[ \t\n]*" address)
>>                               (string-match "[ \t\n]*\\'" address
>>                               (match-end 0))))
>>
>> things work for me.
>
> I don't  think this change is  correct, it doesn't remove  the leading
> whitespace like before.

You are  right here.  But still i  didn't dream the  symptom :)  nor the
backtrace. 

Message #16 received at 32609 <at> debbugs.gnu.org (full text, mbox):

From: Noam Postavsky <npostavs <at> gmail.com>
To: 32609 <at> debbugs.gnu.org
Cc: "Dr. Johannes Brügmann" <dr.johannes.bruegmann <at> gmail.com>
Subject: Re: bug#32609: 26.1; mail-strip-quoted-names:205
Date: Sun, 02 Sep 2018 14:07:51 -0400

tags 32609 + unreproducible
quit

> From: Dr. Johannes Brügmann <dr.johannes.bruegmann <at> gmail.com>

>>> I   tried  to   reply  to   an  email   by  pressing   r,  R,   and  M-x
>>> gnus-article-reply-with-original.   None  of  them  worked.   Minibuffer
>>> displayed  "Mark  Set" but  nothing  happened.  I enabled  Debugging  on
>>> C-g. The *Backtrace*-buffer showed
>>>      string-match("\\`[ \t\n]*" "")
>>>      mail-strip-quoted-names("")
>>>      mail-dont-reply-to("foo <at> bar.com")
>>
>> If I evaluate (mail-dont-reply-to "foo <at> bar.com") it doesn't hang.

> The  mail address  i  wanted  to reply  to  didn't  contain any  special
> characters.  Neither did  the call  to mail-dont-reply-to.  I understand
> that special characters could cause such a behaviour.
>
> As part of following your question, I've put the original file back into
> its place, and  then i recompiled using  byte-compile-file and restarted
> emacs.   Now the  symtom is  no  longer reproducible.  Neither with  the
> originating email, nor with any others that i tested.

I think that means that it's not actually getting stuck in
mail-dont-reply-to, there is some wider loop and you just happened to
stop there.  Next this happens, try to continue a bit and see if you can
find where it loops back again.

Message #21 received at 32609 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Noam Postavsky <npostavs <at> gmail.com>
Cc: 32609 <at> debbugs.gnu.org,
 Dr. Johannes Brügmann <dr.johannes.bruegmann <at> gmail.com>
Subject: Re: bug#32609: 26.1; mail-strip-quoted-names:205
Date: Mon, 23 Sep 2019 17:38:39 +0200

Noam Postavsky <npostavs <at> gmail.com> writes:

> I think that means that it's not actually getting stuck in
> mail-dont-reply-to, there is some wider loop and you just happened to
> stop there.  Next this happens, try to continue a bit and see if you can
> find where it loops back again.

The issue wasn't reproducible, so I'm closing this bug report.  If it
happens again, please reopen.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.