diff options
| author | Alex Musolino <alex@musolino.id.au> | 2020-12-15 20:45:26 +1030 |
|---|---|---|
| committer | Alex Musolino <alex@musolino.id.au> | 2020-12-15 20:45:26 +1030 |
| commit | 404c901f299c4d93cb159a3c44c2977a25408319 (patch) | |
| tree | f29d2a61f872c27f14f33fbf842e6dff885f7dc1 | |
| parent | 81647c202550634b99c2bc93dd055b8d54725abf (diff) | |
| download | plan9front-404c901f299c4d93cb159a3c44c2977a25408319.tar.xz | |
ico: avoid potential nil pointer dereferences
Bgeticon can fail, leaving the Icon img pointers nil.
| -rw-r--r-- | sys/src/cmd/jpg/ico.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/sys/src/cmd/jpg/ico.c b/sys/src/cmd/jpg/ico.c index 7fb270ed3..f8f9c92ff 100644 --- a/sys/src/cmd/jpg/ico.c +++ b/sys/src/cmd/jpg/ico.c @@ -535,6 +535,8 @@ eresized(int new) r.max.x = screen->r.min.x; r.min.y = screen->r.min.y + font->height + 2*BORDER; for(icon = h.first; icon != nil; icon = icon->next){ + if(icon->img == nil) + continue; r.min.x = r.max.x + BORDER; r.max.x = r.min.x + Dx(icon->img->r); r.max.y = r.min.y + Dy(icon->img->r); |