diff src/browser.c @ 79:bfdc427b936d

[gaim-migrate @ 89] I'll save time and just post the email :-) Summary of changes: * Misc malloc/free cleanups, use g_malloc more places and other small stuff (e.g. lineardata not being freed in the error case in sound.c) * Misc signed/unsigned cleanups (use size_t more often) * read() can return -1 at any point, check return values more rigorously (read_rv variables used for this) * In can_play_audio, stat requires a pointer to an allocated stat_buf (the address of an automatic variable) * escape_text needs a buffer at least 4 times the size of the text being passed in (not 2 times); I can force core dumps with lots of newlines otherwise * There's a debug statement in netscape_command (browser.c) that was printf("Hello%d\n"); with no int for the %d; I threw in a getppid(), but the statement should probably come out eventually. Thanks, G Sumner Hayes! committer: Tailor Script <[e-mail]>

--- a/src/browser.c	Fri Mar 31 20:22:12 2000 +0000
+++ b/src/browser.c	Wed Apr 05 05:34:08 2000 +0000
@@ -76,7 +76,7 @@
         if (screen != save_screen) {
                 Display *dpy = DisplayOfScreen(screen);
                 Atom __SWM_VROOT = None;
-                int i;
+                unsigned int i;
                 Window rootReturn, parentReturn, *children;
                 unsigned int numChildren;
 
@@ -409,7 +409,7 @@
         if (!raise_p)
         {
                 char *close;
-                new_command = (char *) malloc (strlen (command) + 20);
+                new_command = g_malloc (strlen (command) + 20);
                 strcpy (new_command, command);
                 close = strrchr (new_command, ')');
                 if (close)
@@ -617,7 +617,7 @@
 			args[0] = g_strdup("netscape");
 			args[1] = NULL;
                         e = execvp(args[0], args);
-                        printf("Hello%d\n");
+                        printf("Hello%d\n", getppid());
                         
 			_exit(0);
 		} else {