mirror of
git://git.code.sf.net/p/zsh/code
synced 2025-10-28 17:10:59 +01:00
25611, 25613: protect against two possible NULL dereferences.
This commit is contained in:
parent
29680850a0
commit
13f60efbd7
2 changed files with 25 additions and 18 deletions
|
|
@ -874,16 +874,17 @@ zccmd_attr(const char *nam, char **args)
|
|||
if ((zca = zcurses_attrget(w->win, ptr)) == NULL) {
|
||||
zwarnnam(nam, "attribute `%s' not known", ptr);
|
||||
ret = 1;
|
||||
}
|
||||
switch(onoff) {
|
||||
case ZCURSES_ATTRON:
|
||||
if (wattron(w->win, zca->number) == ERR)
|
||||
ret = 1;
|
||||
break;
|
||||
case ZCURSES_ATTROFF:
|
||||
if (wattroff(w->win, zca->number) == ERR)
|
||||
ret = 1;
|
||||
break;
|
||||
} else {
|
||||
switch(onoff) {
|
||||
case ZCURSES_ATTRON:
|
||||
if (wattron(w->win, zca->number) == ERR)
|
||||
ret = 1;
|
||||
break;
|
||||
case ZCURSES_ATTROFF:
|
||||
if (wattroff(w->win, zca->number) == ERR)
|
||||
ret = 1;
|
||||
break;
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
@ -948,14 +949,17 @@ zccmd_bg(const char *nam, char **args)
|
|||
if ((zca = zcurses_attrget(w->win, ptr)) == NULL) {
|
||||
zwarnnam(nam, "attribute `%s' not known", ptr);
|
||||
ret = 1;
|
||||
}
|
||||
switch(onoff) {
|
||||
case ZCURSES_ATTRON:
|
||||
ch |= zca->number;
|
||||
break;
|
||||
case ZCURSES_ATTROFF:
|
||||
ch &= ~zca->number;
|
||||
break;
|
||||
} else {
|
||||
switch(onoff) {
|
||||
case ZCURSES_ATTRON:
|
||||
if (wattron(w->win, zca->number) == ERR)
|
||||
ret = 1;
|
||||
break;
|
||||
case ZCURSES_ATTROFF:
|
||||
if (wattroff(w->win, zca->number) == ERR)
|
||||
ret = 1;
|
||||
break;
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue