- add xstrdup to have proper checking for memory allocation failures

[pkg/abook.git] / list.c

diff --git a/list.c b/list.c
index 31853a49e4a716d5539c08816c826919f7723caa..de47d6b419246c5df21ebf85837bd9312e5514ac 100644 (file)
--- a/list.c
+++ b/list.c
@@ -17,6 +17,7 @@
 #include "list.h"
 #include "misc.h"
 #include "options.h"
+#include "xmalloc.h"
 
 #define MIN_EXTRA_COLUMN       ADDRESS /* 2 */
 #define MAX_EXTRA_COLUMN       LAST_FIELD
@@ -32,7 +33,7 @@ extern int items;
 extern list_item *database;
 extern struct abook_field abook_fields[];
 
-WINDOW *list = NULL;
+static WINDOW *list = NULL;
 
 static int
 init_extra_field(enum str_opts option)
@@ -87,7 +88,7 @@ refresh_list()
 
        ui_print_number_of_items();
 
-       if(items < 1) {
+       if(list_is_empty()) {
                refresh();
                wrefresh(list);
                return;
@@ -354,7 +355,7 @@ duplicate_item()
                return 1;
 
        for(i = 0; i < ITEM_FIELDS; i++)
-               item[i] = database[curitem][i] ? strdup(database[curitem][i]) :
+               item[i] = database[curitem][i] ? xstrdup(database[curitem][i]) :
                        NULL;
 
        if(add_item2database(item))