From: Peter Stephenson <pws@csr.com>
To: zsh-workers@sunsite.dk
Subject: Re: zsh 4.3.4 crashes in prompt-related code under Mac OS X
Date: Mon, 22 Oct 2007 11:10:55 +0100 [thread overview]
Message-ID: <200710221010.l9MAAuLw005609@news01.csr.com> (raw)
In-Reply-To: <20071020025428.GR1777@prunille.vinc17.org>
Vincent Lefevre wrote:
> If I understand correctly, the crash occurs here in preprompt():
>
> /* The PROMPT_SP heuristic will move the prompt down to a new line
> * if there was any dangling output on the line (assuming the termina
> l
> * has automatic margins, but we try even if hasam isn't set). */
> char *str;
> int percents = opts[PROMPTPERCENT];
> opts[PROMPTPERCENT] = 1;
> str = promptexpand("%B%S%#%s%b", 0, NULL, NULL);
> opts[PROMPTPERCENT] = percents;
> fprintf(shout, "%s%*s\r", str, (int)columns - 1 - !hasxn, "");
> free(str);
> ^^^^
Thanks for looking...
The obvious cause is some allocation error in promptexpand(). However,
I can't see any sign of it. We allocate 256 bytes initially, which is
bound to be enough for the string being expanded there. (I see one
thing that worried me a bit that I've fixed but I don't think that can
possibly be related.)
The only other possibility I can see locally is that (columns - 1 -
!hasxn) is returning a negative width that's having a bad effect, but
it should simply change the effect of the alignment.
Or maybe "shout" has got into some unusable state. I suppose it's not
unreasonable to test it's not NULL, although it's rather paranoid.
Otherwise it's presumably a knock-on effect from some other part of the
code.
Index: Src/prompt.c
===================================================================
RCS file: /cvsroot/zsh/zsh/Src/prompt.c,v
retrieving revision 1.42
diff -u -r1.42 prompt.c
--- Src/prompt.c 21 Sep 2006 16:36:53 -0000 1.42
+++ Src/prompt.c 22 Oct 2007 10:09:40 -0000
@@ -181,10 +181,10 @@
bp1 = NULL;
truncwidth = 0;
putpromptchar(1, '\0');
- addbufspc(1);
+ addbufspc(2);
if(dontcount)
*bp++ = Outpar;
- *bp = 0;
+ *bp = '\0';
if (!ns) {
/* If zero, Inpar, Outpar and Nularg should be removed. */
for (bp = buf; *bp; ) {
Index: Src/utils.c
===================================================================
RCS file: /cvsroot/zsh/zsh/Src/utils.c,v
retrieving revision 1.167
diff -u -r1.167 utils.c
--- Src/utils.c 5 Sep 2007 16:16:17 -0000 1.167
+++ Src/utils.c 22 Oct 2007 10:09:41 -0000
@@ -1160,7 +1160,7 @@
int period = getiparam("PERIOD");
int mailcheck = getiparam("MAILCHECK");
- if (isset(PROMPTSP) && isset(PROMPTCR) && !use_exit_printed) {
+ if (isset(PROMPTSP) && isset(PROMPTCR) && !use_exit_printed && shout) {
/* The PROMPT_SP heuristic will move the prompt down to a new line
* if there was any dangling output on the line (assuming the terminal
* has automatic margins, but we try even if hasam isn't set).
@@ -1284,7 +1284,7 @@
checkmailpath(arr);
popheap();
}
- } else {
+ } else if (shout) {
if (st.st_size && st.st_atime <= st.st_mtime &&
st.st_mtime > lastmailcheck) {
if (!u) {
@@ -2302,7 +2302,7 @@
if (ask) {
if (noquery(0)) {
x = 'n';
- } else {
+ } else if (shout) {
char *pptbuf;
pptbuf = promptexpand(sprompt, 0, best, guess);
zputs(pptbuf, shout);
--
Peter Stephenson <pws@csr.com> Software Engineer
CSR PLC, Churchill House, Cambridge Business Park, Cowley Road
Cambridge, CB4 0WZ, UK Tel: +44 (0)1223 692070
next prev parent reply other threads:[~2007-10-22 10:12 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-30 8:57 Vincent Lefevre
2007-08-30 15:12 ` Peter Stephenson
2007-10-20 2:54 ` Vincent Lefevre
2007-10-22 10:10 ` Peter Stephenson [this message]
2007-10-23 11:54 ` Vincent Lefevre
2007-10-23 11:58 ` Peter Stephenson
2007-11-01 14:23 ` Vincent Lefevre
2007-11-05 14:43 ` Peter Stephenson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200710221010.l9MAAuLw005609@news01.csr.com \
--to=pws@csr.com \
--cc=zsh-workers@sunsite.dk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).