Check to make sure our call to e_addressbook_model_get_card doesn't return

2001-10-05  Jon Trowbridge  <trow@ximian.com>

	* gui/component/select-names/e-select-names.c
	(real_add_address_cb): Check to make sure our call to
	e_addressbook_model_get_card doesn't return NULL.

	* gui/widgets/e-addressbook-model.c
	(e_addressbook_model_get_card): Chek that we aren't requesting a
	negative row.

	* gui/contact-list-editor/e-contact-list-editor.c (add_email_cb):
	Move to the bottom of the scrolled window, so we can see the
	address we just added.
	(table_drag_data_received_cb): Move to the bottom of the scrolled
	window, so we can see the contact we just dropped.

	* gui/component/addressbook.c (addressbook_factory_new_control):
	We don't own the string returned by e_categories_master_list_nth,
	so terrible things will happen if we free it. (Bug 10916)

svn path=/trunk/; revision=13469

1 files changed, 1 insertions, 1 deletions

diff --git a/addressbook/gui/widgets/e-addressbook-model.c b/addressbook/gui/widgets/e-addressbook-model.c
index d3c59fbcfd..fd67d5fd23 100644
--- a/addressbook/gui/widgets/e-addressbook-model.c
+++ b/addressbook/gui/widgets/e-addressbook-model.c
@@ -419,7 +419,7 @@ ECard *
 e_addressbook_model_get_card(EAddressbookModel *model,
 			     int                row)
 {
-	if (model->data && row < model->data_count) {
+	if (model->data && 0 <= row && row < model->data_count) {
 		ECard *card;
 		card = e_card_duplicate (model->data[row]);
 		return card;