* citadel.c: hitting enter without entering a room name in the <.G>oto and

  <.S>kip commands now does nothing instead of exhibiting undefined behavior

diff --git a/citadel/ChangeLog b/citadel/ChangeLog
index ac2b2ac4c5b017702436e741cc7fd2334c8ee2ff..ea1e43183897fcf46f6c4fd20fa82a9ed9d4c7cf 100644 (file)
--- a/citadel/ChangeLog
+++ b/citadel/ChangeLog
@@ -1,4 +1,8 @@
  $Log$
+ Revision 601.125  2003/01/23 05:50:59  ajc
+ * citadel.c: hitting enter without entering a room name in the <.G>oto and
+   <.S>kip commands now does nothing instead of exhibiting undefined behavior
+
  Revision 601.124  2003/01/23 04:42:33  ajc
  * Began reviewing, updating, and HTML-izing the documentation
 
@@ -4443,3 +4447,4 @@ Sat Jul 11 00:20:48 EDT 1998 Nathan Bryant <bryant@cs.usm.maine.edu>
 
 Fri Jul 10 1998 Art Cancro <ajc@uncensored.citadel.org>
        * Initial CVS import
+

diff --git a/citadel/citadel.c b/citadel/citadel.c
index db25b2d17d08aedfcf99b4a19ca5923fd62c3604..2e28ef9d9894a567f17163278c568ea0de14c1ff 100644 (file)
--- a/citadel/citadel.c
+++ b/citadel/citadel.c
@@ -1323,13 +1323,18 @@ NEWUSR: if (strlen(rc_password) == 0) {
                                dotgoto(ipc, "_MAIL_", 1, 0);
                                break;
                        case 20:
-                               updatels(ipc);
-                               dotgoto(ipc, argbuf, 0, 0);
+                               if (strlen(argbuf) > 0) {
+                                       updatels(ipc);
+                                       dotgoto(ipc, argbuf, 0, 0);
+                               }
                                break;
                        case 52:
-                               if (rc_alt_semantics)
-                                       updatelsa(ipc);
-                               dotgoto(ipc, argbuf, 0, 0);
+                               if (strlen(argbuf) > 0) {
+                                       if (rc_alt_semantics) {
+                                               updatelsa(ipc);
+                                       }
+                                       dotgoto(ipc, argbuf, 0, 0);
+                               }
                                break;
                        case 95: /* what exactly is the numbering scheme supposed to be anyway? */
                                dotungoto(ipc, argbuf);