* Re: core dump with arithmetic assignment
@ 1999-10-15 6:30 Sven Wischnowsky
0 siblings, 0 replies; 2+ messages in thread
From: Sven Wischnowsky @ 1999-10-15 6:30 UTC (permalink / raw)
To: zsh-workers
Adam Spiers wrote:
> I haven't recompiled for quite a few patches now, so this may already
> be fixed.
>
> % (( 1 = 1 )) && echo hi
> zsh: lvalue required
> zsh: 11025 segmentation fault (core dumped) /bin/zsh
This at least avoids the SEGV and probably is even the correct fix. At
least it seems strange to first test for `if (!s)', print the error if
that is true and later try `setnparam(s, v)'. And returning the
value built after the `zerr()' seems ok, too. And `noeval' seems to be
used for something else.
Bye
Sven
--- oldsrc/math.c Thu Oct 14 16:26:25 1999
+++ Src/math.c Fri Oct 15 08:26:57 1999
@@ -467,6 +467,7 @@
zerr("lvalue required", NULL, 0);
v.type = MN_INTEGER;
v.u.l = 0;
+ return v;
}
if (noeval)
return v;
--
Sven Wischnowsky wischnow@informatik.hu-berlin.de
^ permalink raw reply [flat|nested] 2+ messages in thread
* core dump with arithmetic assignment
@ 1999-10-14 15:58 Adam Spiers
0 siblings, 0 replies; 2+ messages in thread
From: Adam Spiers @ 1999-10-14 15:58 UTC (permalink / raw)
To: zsh workers mailing list
I haven't recompiled for quite a few patches now, so this may already
be fixed.
% (( 1 = 1 )) && echo hi
zsh: lvalue required
zsh: 11025 segmentation fault (core dumped) /bin/zsh
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~1999-10-15 6:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
1999-10-15 6:30 core dump with arithmetic assignment Sven Wischnowsky
-- strict thread matches above, loose matches on Subject: below --
1999-10-14 15:58 Adam Spiers
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).