GNU bug report logs - #14412
24.3.50; emacs_backtrace.txt

Previous Next

Package: emacs;

Reported by: "Drew Adams" <drew.adams <at> oracle.com>

Date: Thu, 16 May 2013 21:22:01 UTC

Severity: normal

Tags: moreinfo

Found in version 24.3.50

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

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

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

View this report as an mbox folder, status mbox, maintainer mbox

Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#14412; Package emacs. (Thu, 16 May 2013 21:22:01 GMT) Full text and rfc822 format available.
Acknowledgement sent to "Drew Adams" <drew.adams <at> oracle.com>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Thu, 16 May 2013 21:22:01 GMT) Full text and rfc822 format available.

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

From: "Drew Adams" <drew.adams <at> oracle.com>
To: <bug-gnu-emacs <at> gnu.org>
Subject: 24.3.50; emacs_backtrace.txt
Date: Thu, 16 May 2013 14:21:05 -0700

Backtrace:
0x01159525
0x01159597
0x01001459
0x01021A5E
0x0101675C
0x010D82B4
0x010D938E
0x010D8F02
0x010116DA
0x010118DF
0x0101120C
0x01011326
0x01011365
0x01006765
0x010E4AC8
0x0101596C
0x01014E87
0x010E5146
0x01015511
0x01014E87
0x010E5146
0x0101596C
0x01014E87
0x010E5146
0x0101596C
0x01014E87
0x01013C70
0x010141D4
0x010E8FBA
0x01014B45
0x010E5146
0x01015511
0x01014E87
0x01014229
0x01024826
0x01010C39
0x01023814
0x01010696
0x010237CE
0x01022D88
0x010230B5
0x010029C4
0x010010F9
0x7C81776B
 
 
 

In GNU Emacs 24.3.50.1 (i386-mingw-nt5.1.2600)
 of 2013-05-14 on ODIEONE
Bzr revision: 112586 juri <at> jurta.org-20130514233814-nnkh1ymiqgoq2fk6
Windowing system distributor `Microsoft Corp.', version 5.1.2600
Configured using:
 `configure --with-gcc (4.7) --no-opt --enable-checking --cflags
 -IC:/Devel/emacs/build/include --ldflags -LC:/Devel/emacs/build/lib'
Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#14412; Package emacs. (Fri, 17 May 2013 10:01:01 GMT) Full text and rfc822 format available.

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Drew Adams <drew.adams <at> oracle.com>
Cc: 14412 <at> debbugs.gnu.org
Subject: Re: bug#14412: 24.3.50; emacs_backtrace.txt
Date: Fri, 17 May 2013 12:59:25 +0300

> From: "Drew Adams" <drew.adams <at> oracle.com>
> Date: Thu, 16 May 2013 14:21:05 -0700
> 
> Backtrace:
> 0x01159525
> 0x01159597
> 0x01001459
> 0x01021A5E
> 0x0101675C
> 0x010D82B4
> 0x010D938E
> 0x010D8F02
> 0x010116DA
> 0x010118DF
> 0x0101120C
> 0x01011326
> 0x01011365
> 0x01006765
> 0x010E4AC8
> 0x0101596C
> 0x01014E87
> 0x010E5146
> 0x01015511
> 0x01014E87
> 0x010E5146
> 0x0101596C
> 0x01014E87
> 0x010E5146
> 0x0101596C
> 0x01014E87
> 0x01013C70
> 0x010141D4
> 0x010E8FBA
> 0x01014B45
> 0x010E5146
> 0x01015511
> 0x01014E87
> 0x01014229
> 0x01024826
> 0x01010C39
> 0x01023814
> 0x01010696
> 0x010237CE
> 0x01022D88
> 0x010230B5
> 0x010029C4
> 0x010010F9
> 0x7C81776B

  w32_backtrace at C:\Devel\emacs\repo\build\src/w32fns.c:7740
  emacs_abort at C:\Devel\emacs\repo\build\src/w32fns.c:7772
  terminate_due_to_signal at C:\Devel\emacs\repo\build\src/emacs.c:343
  die at C:\Devel\emacs\repo\build\src/alloc.c:6522
  unbind_to at C:\Devel\emacs\repo\build\src/eval.c:3124
  Fprinc at C:\Devel\emacs\repo\build\src/print.c:658
  print_error_message at C:\Devel\emacs\repo\build\src/print.c:901
  Ferror_message_string at C:\Devel\emacs\repo\build\src/print.c:826
  skip_debugger at C:\Devel\emacs\repo\build\src/eval.c:1566
  maybe_call_debugger at C:\Devel\emacs\repo\build\src/eval.c:1607
  Fsignal at C:\Devel\emacs\repo\build\src/eval.c:1431
  xsignal at C:\Devel\emacs\repo\build\src/eval.c:1466
  xsignal1 at C:\Devel\emacs\repo\build\src/eval.c:1481
  Fsymbol_value at C:\Devel\emacs\repo\build\src/data.c:1061
  exec_byte_code at C:\Devel\emacs\repo\build\src/bytecode.c:717
  funcall_lambda at C:\Devel\emacs\repo\build\src/eval.c:2906
  Ffuncall at C:\Devel\emacs\repo\build\src/eval.c:2723
  exec_byte_code at C:\Devel\emacs\repo\build\src/bytecode.c:900
  funcall_lambda at C:\Devel\emacs\repo\build\src/eval.c:2840
  Ffuncall at C:\Devel\emacs\repo\build\src/eval.c:2723
  exec_byte_code at C:\Devel\emacs\repo\build\src/bytecode.c:900
  funcall_lambda at C:\Devel\emacs\repo\build\src/eval.c:2906
  Ffuncall at C:\Devel\emacs\repo\build\src/eval.c:2723
  exec_byte_code at C:\Devel\emacs\repo\build\src/bytecode.c:900
  funcall_lambda at C:\Devel\emacs\repo\build\src/eval.c:2906
  Ffuncall at C:\Devel\emacs\repo\build\src/eval.c:2723
  Fapply at C:\Devel\emacs\repo\build\src/eval.c:2208
  apply1 at C:\Devel\emacs\repo\build\src/eval.c:2442
  Fcall_interactively at C:\Devel\emacs\repo\build\src/callint.c:381
  Ffuncall at C:\Devel\emacs\repo\build\src/eval.c:2681
  exec_byte_code at C:\Devel\emacs\repo\build\src/bytecode.c:900
  funcall_lambda at C:\Devel\emacs\repo\build\src/eval.c:2840
  Ffuncall at C:\Devel\emacs\repo\build\src/eval.c:2723
  call1 at C:\Devel\emacs\repo\build\src/eval.c:2468
  command_loop_1 at C:\Devel\emacs\repo\build\src/keyboard.c:1578
  internal_condition_case at C:\Devel\emacs\repo\build\src/eval.c:1193
  command_loop_2 at C:\Devel\emacs\repo\build\src/keyboard.c:1167
  internal_catch at C:\Devel\emacs\repo\build\src/eval.c:964
  command_loop at C:\Devel\emacs\repo\build\src/keyboard.c:1146
  recursive_edit_1 at C:\Devel\emacs\repo\build\src/keyboard.c:779
  Frecursive_edit at C:\Devel\emacs\repo\build\src/keyboard.c:843
  main at C:\Devel\emacs\repo\build\src/emacs.c:1531
  ?? at crt1.c:0

It aborts here:

      /* If variable has a trivial value (no forwarding), we can
	 just set it.  No need to check for constant symbols here,
	 since that was already done by specbind.  */
 >>>  else if (XSYMBOL (this_binding.symbol)->redirect == SYMBOL_PLAINVAL)
	SET_SYMBOL_VAL (XSYMBOL (this_binding.symbol),
			this_binding.old_value);

Should we explicitly test for SYMBOLP before calling XSYMBOL?
Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#14412; Package emacs. (Sat, 26 Dec 2015 00:49:02 GMT) Full text and rfc822 format available.

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 14412 <at> debbugs.gnu.org, Drew Adams <drew.adams <at> oracle.com>
Subject: Re: bug#14412: 24.3.50; emacs_backtrace.txt
Date: Sat, 26 Dec 2015 01:47:50 +0100

Eli Zaretskii <eliz <at> gnu.org> writes:

> It aborts here:
>
>       /* If variable has a trivial value (no forwarding), we can
> 	 just set it.  No need to check for constant symbols here,
> 	 since that was already done by specbind.  */
>  >>>  else if (XSYMBOL (this_binding.symbol)->redirect == SYMBOL_PLAINVAL)
> 	SET_SYMBOL_VAL (XSYMBOL (this_binding.symbol),
> 			this_binding.old_value);
>
> Should we explicitly test for SYMBOLP before calling XSYMBOL?

Was this resolved, by any chance?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no
Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#14412; Package emacs. (Sat, 26 Dec 2015 10:07:01 GMT) Full text and rfc822 format available.

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 14412 <at> debbugs.gnu.org, drew.adams <at> oracle.com
Subject: Re: bug#14412: 24.3.50; emacs_backtrace.txt
Date: Sat, 26 Dec 2015 12:07:19 +0200

> From: Lars Ingebrigtsen <larsi <at> gnus.org>
> Cc: Drew Adams <drew.adams <at> oracle.com>,  14412 <at> debbugs.gnu.org
> Date: Sat, 26 Dec 2015 01:47:50 +0100
> 
> Eli Zaretskii <eliz <at> gnu.org> writes:
> 
> > It aborts here:
> >
> >       /* If variable has a trivial value (no forwarding), we can
> > 	 just set it.  No need to check for constant symbols here,
> > 	 since that was already done by specbind.  */
> >  >>>  else if (XSYMBOL (this_binding.symbol)->redirect == SYMBOL_PLAINVAL)
> > 	SET_SYMBOL_VAL (XSYMBOL (this_binding.symbol),
> > 			this_binding.old_value);
> >
> > Should we explicitly test for SYMBOLP before calling XSYMBOL?
> 
> Was this resolved, by any chance?

Not that I could see, but adding the test is trivial, so I will do it
soon.
Reply sent to Eli Zaretskii <eliz <at> gnu.org>:
You have taken responsibility. (Sat, 26 Dec 2015 10:45:02 GMT) Full text and rfc822 format available.
Notification sent to "Drew Adams" <drew.adams <at> oracle.com>:
bug acknowledged by developer. (Sat, 26 Dec 2015 10:45:02 GMT) Full text and rfc822 format available.

Message #19 received at 14412-done <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: larsi <at> gnus.org
Cc: 14412-done <at> debbugs.gnu.org
Subject: Re: bug#14412: 24.3.50; emacs_backtrace.txt
Date: Sat, 26 Dec 2015 12:44:28 +0200

> Date: Sat, 26 Dec 2015 12:07:19 +0200
> From: Eli Zaretskii <eliz <at> gnu.org>
> Cc: 14412 <at> debbugs.gnu.org
> 
> > From: Lars Ingebrigtsen <larsi <at> gnus.org>
> > Cc: Drew Adams <drew.adams <at> oracle.com>,  14412 <at> debbugs.gnu.org
> > Date: Sat, 26 Dec 2015 01:47:50 +0100
> > 
> > Eli Zaretskii <eliz <at> gnu.org> writes:
> > 
> > > It aborts here:
> > >
> > >       /* If variable has a trivial value (no forwarding), we can
> > > 	 just set it.  No need to check for constant symbols here,
> > > 	 since that was already done by specbind.  */
> > >  >>>  else if (XSYMBOL (this_binding.symbol)->redirect == SYMBOL_PLAINVAL)
> > > 	SET_SYMBOL_VAL (XSYMBOL (this_binding.symbol),
> > > 			this_binding.old_value);
> > >
> > > Should we explicitly test for SYMBOLP before calling XSYMBOL?
> > 
> > Was this resolved, by any chance?
> 
> Not that I could see, but adding the test is trivial, so I will do it
> soon.

Done.
Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#14412; Package emacs. (Sat, 26 Dec 2015 17:44:02 GMT) Full text and rfc822 format available.

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

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: Lars Ingebrigtsen <larsi <at> gnus.org>, 14412 <at> debbugs.gnu.org,
 Drew Adams <drew.adams <at> oracle.com>
Subject: Re: bug#14412: 24.3.50; emacs_backtrace.txt
Date: Sat, 26 Dec 2015 09:43:25 -0800

[Message part 1 (text/plain, inline)]
Eli, thanks for fixing that. There's parallel code in backtrace_eval_unrewind 
that presumably should be fixed in a similar way. Also, it's clearer to keep the 
FALLTHROUGH!! comment at the actual fallthrough point. I installed the attached 
additional patch to try to address these issues.

[0001-Propagate-Bug-14412-fix-to-backtrace_eval_unrewind.patch (text/x-diff, attachment)]
bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Sun, 24 Jan 2016 12:24:04 GMT) Full text and rfc822 format available.
This bug report was last modified 9 years and 205 days ago.

Previous Next

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