diff --git a/ChangeLog b/ChangeLog
index 78334a5a2a..ee1ee5074e 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2005-09-02  Bill Nottingham  <notting@redhat.com>
+
+	* src/backends/NetworkManagerRedHat.c: use nm_warning, not nm_error
+
 2005-09-01  Dan Williams  <dcbw@redhat.com>
 
 	* src/NetworkManager.c
diff --git a/gnome/applet/applet-dbus-devices.c b/gnome/applet/applet-dbus-devices.c
index 44ba2914d6..038d47153e 100644
--- a/gnome/applet/applet-dbus-devices.c
+++ b/gnome/applet/applet-dbus-devices.c
@@ -847,8 +847,9 @@ static void nmwa_dbus_update_dialup_cb (DBusPendingCall *pcall, void *user_data)
 			applet->dialup_list = NULL;
 		}
 
-		for (item = dialup_devices; *item; item++)
+		for (item = dialup_devices; *item; item++) {
 			applet->dialup_list = g_slist_append (applet->dialup_list, g_strdup (*item));
+		}
 
 		dbus_free_string_array (dialup_devices);
 	}
diff --git a/src/backends/NetworkManagerRedHat.c b/src/backends/NetworkManagerRedHat.c
index c40cab33d7..8152c67376 100644
--- a/src/backends/NetworkManagerRedHat.c
+++ b/src/backends/NetworkManagerRedHat.c
@@ -696,7 +696,7 @@ gboolean nm_system_activate_dialup (GSList *list, const char *dialup)
 				ret = TRUE;
 			} else {
 				/* FIXME: Decode errors into something sensible */
-				nm_error ("Couldn't activate dialup device %s (%s) - %d", dialup, (char *) config->data, status);
+				nm_warning ("Couldn't activate dialup device %s (%s) - %d", dialup, (char *) config->data, status);
 				ret = FALSE;
 			}
 			break;