stop using bzero() and check a few return values

3 files changed, 9 insertions, 8 deletions

diff --git a/cmd.c b/cmd.c
index edd6d00..551a069 100644
--- a/cmd.c
+++ b/cmd.c
@@ -339,7 +339,7 @@ static void cmd_module __P1 (char *,arg) {
 	/* I changed it to work this way so that you can have libs in multiple places and
 	 * also eventually to allow it to use .dll instead of .so under the cygwin environment */
 	for( pindex = 0; pindex < 5; pindex++ ) {
-		bzero( libname, 1024 );
+		memset( libname, 0, sizeof libname );
 
         /* don't look for name without .so, it breaks if you have a file
          * with the same name in the current dir and making it .so for sure
@@ -574,9 +574,11 @@ static void cmd_shell __P1 (char *,arg)
         }
     } else {
         tty_quit();
-	
-        system(arg);
-	
+
+        if (system(arg) == -1) {
+            perror("system()");
+        }
+
         tty_start();
         tty_gotoxy(col0 = 0, line0 = lines -1);
         tty_puts(tty_clreoln);
diff --git a/configure.in b/configure.in
index c55310c..762275e 100644
--- a/configure.in
+++ b/configure.in
@@ -112,7 +112,6 @@ AC_C_CONST
 
 # Checks for library functions.
 AC_FUNC_MALLOC
-AC_CHECK_FUNCS([bzero])
 
 AC_OUTPUT(Makefile)
 
diff --git a/powwow-muc.c b/powwow-muc.c
index c98547e..4f8ae18 100644
--- a/powwow-muc.c
+++ b/powwow-muc.c
@@ -82,8 +82,8 @@ int main( int argc, char *argv[] ) {
 	timeout( 0 );
 	looping = 1;
 	while( looping ) {
-		bzero( line, 1000 );
-		fgets( line, 1000, in );
+		memset( line, 0, sizeof line );
+		fgets( line, sizeof line, in );
 
 		/* get file pos */
 		new_pos = curr_pos = ftell( in );
@@ -284,7 +284,7 @@ int main( int argc, char *argv[] ) {
 
 			/* read to next newline so we don't break up
 			   lines seeking around */
-			fgets( line, 1000, in );
+			fgets( line, sizeof line, in );
 			new_pos = ftell( in );
 
 			/* Make a note of moving */