[hackers] [sent] fix 2 uninitialized variables and bin == NULL check || Markus Teich

From: <git_AT_suckless.org>
Date: Wed, 9 Dec 2015 00:48:05 +0100 (CET)

commit 008ff1b88b3e9cdd13e82d0ec4fb573c977f6a3d
Author: Markus Teich <markus.teich_AT_stusta.mhn.de>
AuthorDate: Wed Dec 9 00:50:05 2015 +0100
Commit: Markus Teich <markus.teich_AT_stusta.mhn.de>
CommitDate: Wed Dec 9 00:50:05 2015 +0100

    fix 2 uninitialized variables and bin == NULL check

diff --git a/drw.c b/drw.c
index 15797ce..be30400 100644
--- a/drw.c
+++ b/drw.c
_AT_@ -228,7 +228,7 @@ int
 drw_text(Drw *drw, int x, int y, unsigned int w, unsigned int h, const char *text, int invert) {
         char buf[1024];
         int tx, ty, th;
- unsigned int ew;
+ unsigned int ew = 0;
         Colormap cmap;
         Visual *vis;
         XftDraw *d;
diff --git a/sent.c b/sent.c
index 5c72597..51f6eed 100644
--- a/sent.c
+++ b/sent.c
_AT_@ -162,23 +162,25 @@ filter(int fd, const char *cmd)
 Image *ffopen(char *filename)
 {
         unsigned char hdr[16];
- char *bin;
+ char *bin = NULL;
         regex_t regex;
         Image *img;
         size_t i;
         int tmpfd, fd;
 
- for (bin = NULL, i = 0; i < LEN(filters); i++) {
+ for (i = 0; i < LEN(filters); i++) {
                 if (regcomp(&regex, filters[i].regex,
                             REG_NOSUB | REG_EXTENDED | REG_ICASE))
                         continue;
                 if (!regexec(&regex, filename, 0, NULL, 0)) {
- if (!(bin = filters[i].bin))
- return NULL;
+ bin = filters[i].bin
                         break;
                 }
         }
 
+ if (!bin)
+ return NULL;
+
         if ((fd = open(filename, O_RDONLY)) < 0) {
                 eprintf("Unable to open file %s:", filename);
                 return NULL;
_AT_@ -355,7 +357,7 @@ void ffdraw(Image *img)
 void getfontsize(Slide *s, unsigned int *width, unsigned int *height)
 {
         int i, j;
- unsigned int curw, imax;
+ unsigned int curw, new_max;
         float lfac = linespacing * (s->linecount - 1) + 1;
 
         /* fit height */
_AT_@ -369,13 +371,12 @@ void getfontsize(Slide *s, unsigned int *width, unsigned int *height)
         *width = 0;
         for (i = 0; i < s->linecount; i++) {
                 curw = drw_fontset_getwidth(d, s->lines[i]);
- if (curw >= *width)
- imax = i;
+ newmax = (curw >= *width);
                 while (j > 0 && curw > xw.uw) {
                         drw_setfontset(d, fonts[--j]);
                         curw = drw_fontset_getwidth(d, s->lines[i]);
                 }
- if (imax == i)
+ if (newmax)
                         *width = curw;
         }
         *height = fonts[j]->h * lfac;
Received on Wed Dec 09 2015 - 00:48:05 CET

This archive was generated by hypermail 2.3.0 : Wed Dec 09 2015 - 01:00:13 CET