diff options
author | Rob Landley <rob@landley.net> | 2005-09-14 19:28:15 +0400 |
---|---|---|
committer | Rob Landley <rob@landley.net> | 2005-09-14 19:28:15 +0400 |
commit | b7285001c9f21bd2971f9fd511b229e1f0da541c (patch) | |
tree | a8254e4f52edcdd333bf082477cc80449a61594b /util-linux/readprofile.c | |
parent | 7573ac6edf19ed2ea39fcd420b80ee05e8796e1c (diff) |
Cleanup patch from Tito.
Diffstat (limited to 'util-linux/readprofile.c')
-rw-r--r-- | util-linux/readprofile.c | 21 |
1 files changed, 9 insertions, 12 deletions
diff --git a/util-linux/readprofile.c b/util-linux/readprofile.c index 95635801f..5144de711 100644 --- a/util-linux/readprofile.c +++ b/util-linux/readprofile.c @@ -138,8 +138,6 @@ int readprofile_main(int argc, char **argv) } fd = bb_xopen(defaultpro,O_WRONLY); - if (fd < 0) - bb_perror_msg_and_die(defaultpro); if (write(fd, &multiplier, to_write) != to_write) bb_perror_msg_and_die("error writing %s", defaultpro); @@ -151,13 +149,14 @@ int readprofile_main(int argc, char **argv) /* * Use an fd for the profiling buffer, to skip stdio overhead */ - if (((proFd = bb_xopen(proFile,O_RDONLY)) < 0) - || ((int)(len=lseek(proFd,0,SEEK_END)) < 0) + + proFd = bb_xopen(proFile,O_RDONLY); + + if (((int)(len=lseek(proFd,0,SEEK_END)) < 0) || (lseek(proFd,0,SEEK_SET) < 0)) bb_perror_msg_and_die(proFile); - if (!(buf = xmalloc(len))) - bb_perror_nomsg_and_die(); + buf = xmalloc(len); if (read(proFd,buf,len) != len) bb_perror_msg_and_die(proFile); @@ -176,8 +175,8 @@ int readprofile_main(int argc, char **argv) small++; } if (big > small) { - fprintf(stderr,"Assuming reversed byte order. " - "Use -n to force native byte order.\n"); + bb_error_msg("Assuming reversed byte order. " + "Use -n to force native byte order."); for (p = buf; p < buf+entries; p++) for (i = 0; i < sizeof(*buf)/2; i++) { unsigned char *b = (unsigned char *) p; @@ -199,8 +198,6 @@ int readprofile_main(int argc, char **argv) total = 0; map = bb_xfopen(mapFile, "r"); - if (map == NULL) - bb_perror_msg_and_die(mapFile); while (fgets(mapline,S_LEN,map)) { if (sscanf(mapline,"%llx %s %s",&fn_add,mode,fn_name) != 3) @@ -215,7 +212,7 @@ int readprofile_main(int argc, char **argv) } if (!add0) - bb_error_msg_and_die("can't find \"_stext\" in %s\n", mapFile); + bb_error_msg_and_die("can't find \"_stext\" in %s", mapFile); /* * Main loop. @@ -224,7 +221,7 @@ int readprofile_main(int argc, char **argv) unsigned int this = 0; if (sscanf(mapline,"%llx %s %s",&next_add,mode,next_name) != 3) - bb_error_msg_and_die("%s(%i): wrong map line\n", + bb_error_msg_and_die("%s(%i): wrong map line", mapFile, maplineno); header_printed = 0; |