Advertisement
Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- Index: tools/splitshp.cc
- ===================================================================
- --- tools/splitshp.cc (revision 7169)
- +++ tools/splitshp.cc (working copy)
- @@ -20,6 +20,7 @@
- # include <config.h>
- #endif
- +#include <cassert>
- #include <iostream>
- #include <cstdlib>
- @@ -129,6 +130,7 @@
- uint8 *data;
- char *framename;
- int i;
- + size_t err;
- shpfile = fopen (filename, "rb");
- if (!shpfile) {
- @@ -150,7 +152,8 @@
- framename = framefilename(filename, i);
- cout << "writing " << framename << "..." << endl;
- framefile = fopen(framename, "wb");
- - fread(data, 1, 64, shpfile);
- + err = fread(data, 1, 64, shpfile);
- + assert (err == 64);
- fwrite(data, 1, 64, framefile);
- fclose(framefile);
- delete[] framename;
- @@ -183,7 +186,8 @@
- write4(framefile, 8);
- data = new uint8[datalen];
- - fread(data, 1, datalen, shpfile);
- + err = fread(data, 1, datalen, shpfile);
- + assert (err == datalen);
- fwrite(data, 1, datalen, framefile);
- fclose(framefile);
- delete[] framename;
- @@ -203,6 +207,7 @@
- int file_size, shape_size, hdr_size, frame_size;
- int total_size;
- uint8 *data;
- + size_t err;
- shpfile = fopen(shapefile, "wb");
- @@ -231,7 +236,8 @@
- data = new uint8[64];
- - fread(data, 1, 64, framefile);
- + err = fread(data, 1, 64, framefile);
- + assert (err == 64);
- fwrite(data, 1, 64, shpfile);
- fclose(framefile);
- @@ -253,7 +259,8 @@
- data = new uint8[frame_size];
- fseek(framefile, hdr_size, SEEK_SET);
- - fread(data, 1, frame_size, framefile);
- + err = fread(data, 1, frame_size, framefile);
- + assert (err == frame_size);
- fclose(framefile);
- fseek(shpfile, 4 + (i*4), SEEK_SET);
- Index: tools/expack.cc
- ===================================================================
- --- tools/expack.cc (revision 7169)
- +++ tools/expack.cc (working copy)
- @@ -51,7 +51,6 @@
- using std::size_t;
- using std::strrchr;
- using std::strncpy;
- -using std::strncat;
- using std::strlen;
- using std::vector;
- using std::string;
- @@ -269,7 +268,7 @@
- hname[0] = 0;
- if(argc>2) {
- - strncpy(fname, argv[2], 1024);
- + strncpy(fname, argv[2], sizeof (fname));
- if((argv[1][0]=='-')&&(strlen(argv[1])==2)) {
- switch(argv[1][1]) {
- case 'i':
- @@ -295,23 +294,20 @@
- }
- // Read the output file name
- - getline(respfile, temp, 1024);
- - strncpy(fname, path_prefix, 1024);
- - strncat(fname, temp, 1024);
- + getline(respfile, temp, sizeof (temp));
- + snprintf(fname, sizeof (fname), "%s%s", path_prefix, temp);
- // Header file name
- - strncpy (hprefix, temp, 1024);
- + strncpy (hprefix, temp, sizeof (hprefix));
- make_header_name(hprefix);
- - strncpy (hname, path_prefix, 1024);
- - strncat (hname, hprefix, 1024);
- - strncat (hname, ".h", 1024);
- + snprintf (hname, sizeof (hname), "%s%s.h", path_prefix, hprefix);
- strip_path (hprefix);
- make_uppercase (hprefix);
- unsigned int shnum = 0;
- int linenum = 2;
- while(respfile.good()) {
- - getline(respfile, temp, 1024);
- + getline(respfile, temp, sizeof (temp));
- if(strlen(temp)>0) {
- char *ptr = temp;
- if (*ptr == ':')
- @@ -329,8 +325,7 @@
- ptr++;
- }
- char temp2[1024];
- - strncpy(temp2, path_prefix,1024);
- - strncat(temp2, ptr, 1024);
- + snprintf (temp2, sizeof (temp2), "%s%s", path_prefix, ptr);
- if (shnum >= file_names.size())
- file_names.resize(shnum + 1);
- file_names[shnum] = temp2;
- @@ -431,10 +426,9 @@
- char hline[1024];
- ofstream header;
- if (!hname[0]) { // Need header name.
- - strncpy (hprefix, fname, 1024);
- + strncpy (hprefix, fname, sizeof (hprefix));
- make_header_name(hprefix);
- - strncat (hname, hprefix, 1024);
- - strncat (hname, ".h", 1024);
- + snprintf (hname, sizeof (hname), "%s.h", hprefix);
- }
- try {
- U7open(header, hname, true);
- @@ -472,7 +466,7 @@
- delete [] buf;
- infile.close();
- - strncpy (hline, file_names[i].c_str(), 1024);
- + strncpy (hline, file_names[i].c_str(), sizeof (hline));
- strip_path(hline);
- make_header_name(hline);
- make_uppercase(hline);
- Index: tools/ucdump.c
- ===================================================================
- --- tools/ucdump.c (revision 7169)
- +++ tools/ucdump.c (working copy)
- @@ -81,6 +81,7 @@
- ( x - 933 ) / 10, ( y - 1134 ) / 10
- */
- +#include <assert.h>
- #include <stdio.h>
- #include <stdlib.h>
- #include <string.h>
- @@ -859,11 +860,15 @@
- unsigned char* p;
- unsigned char* pp;
- unsigned char* tempstr;
- + size_t err;
- +
- /* Allocate a temporary buffer */
- tempstr = malloc(70 + 1);
- pos = ftell(f);
- pp = p = malloc(ds);
- - fread(p, 1, ds, f);
- + err = fread(p, 1, ds, f);
- + assert (err == ds);
- +
- fseek(f, pos, SEEK_SET);
- /* Print all strings & their offsets */
- while( off < ds )
- @@ -1092,14 +1097,20 @@
- unsigned char* pp;
- unsigned char* pdata;
- unsigned short* pextern;
- + size_t err;
- +
- pos = ftell(f);
- size = s - ds - sizeof(unsigned short);
- pp = p = malloc(size);
- pdata = malloc(ds);
- - fread(pdata, 1, ds, f);
- + err = fread(pdata, 1, ds, f);
- + assert (err == ds);
- +
- if( !mute )
- printf("Code segment at file offset %08lXH\n", ftell(f));
- - fread(p, 1, size, f);
- + err = fread(p, 1, size, f);
- + assert (err == size);
- +
- fseek(f, pos, SEEK_SET);
- /* Print code segment header */
- if( size < 3 * sizeof(unsigned short) )
- @@ -1160,14 +1171,22 @@
- {
- unsigned short s, ds, funcnum;
- long off, bodyoff;
- + size_t err;
- +
- /* Save start offset */
- off = ftell(f);
- /* Read function header */
- - fread(&funcnum, sizeof(unsigned short), 1, f);
- - fread(&s, sizeof(unsigned short), 1, f);
- + err = fread(&funcnum, sizeof(unsigned short), 1, f);
- + assert (err == 1);
- +
- + err = fread(&s, sizeof(unsigned short), 1, f);
- + assert (err == 1);
- +
- /* Save body offset */
- bodyoff = ftell(f);
- - fread(&ds, sizeof(unsigned short), 1, f);
- + err = fread(&ds, sizeof(unsigned short), 1, f);
- + assert (err == 1);
- +
- if( ( ( func == -1 ) || scan_mode ) && ( opcode == -1 ) && ( intrinsic == -1 ) )
- /* Only for general list & scan mode */
- printf("\tFunction #%d (%04XH), offset = %08lx, size = %04x, data = %04x\n", i,
- Index: tools/shp2pcx.cc
- ===================================================================
- --- tools/shp2pcx.cc (revision 7169)
- +++ tools/shp2pcx.cc (working copy)
- @@ -23,6 +23,7 @@
- #include "sdl-compat.h"
- #include SDL_BYTEORDERH
- +#include <cassert>
- #include <iostream>
- #ifndef ALPHA_LINUX_CXX
- @@ -145,6 +146,7 @@
- uint8 block;
- uint8 pix;
- int i, j;
- + size_t err;
- int temp_int;
- @@ -176,7 +178,8 @@
- frame->leftX = 0;
- frame->leftY = 0;
- frame->pixels = new uint8[64];
- - fread(frame->pixels, 1, 64, fp);
- + err = fread(frame->pixels, 1, 64, fp);
- + assert (err == 64);
- }
- } else {
- hdr_size = read4(fp);
- Index: tools/rip.c
- ===================================================================
- --- tools/rip.c (revision 7169)
- +++ tools/rip.c (working copy)
- @@ -1,3 +1,4 @@
- +#include <assert.h>
- #include <stdio.h>
- #include <stdlib.h>
- #include <string.h>
- @@ -8,6 +9,7 @@
- char s[10];
- char filename[18];
- char *pos;
- + char *err;
- FILE *fi=fopen("index","r"),*fi2,*fo=fopen("usecode","wb");
- if (fi==NULL)
- {
- @@ -20,7 +22,9 @@
- exit(0);
- }
- while (!feof(fi)) {
- - fgets(s,10,fi);
- + err = fgets(s,10,fi);
- + assert (err != NULL);
- +
- strcpy(filename, s);
- pos = strchr(filename, '\n');
- if (pos) *pos = '\0';
- @@ -58,6 +62,8 @@
- char s[10];
- char filename[18];
- FILE *fi,*fo,*fo2;
- + size_t err;
- +
- printf("Wody's Rip v0.005\nCopyright (c) 1999 Wody Dragon (a.k.a. Wouter Dijkslag)\n");
- if (argc<2||(!strcasecmp(argv[1],"put")&&argc!=3))
- {
- @@ -96,11 +102,14 @@
- if (fread(&fn,2,1,fi)!=1) break;
- if (fn == 0xFFFF) {
- extended = 1;
- - fread(&fn,2,1,fi);
- - fread(&fs,4,1,fi);
- + err = fread(&fn,2,1,fi);
- + assert (err == 1);
- + err = fread(&fs,4,1,fi);
- + assert (err == 1);
- } else {
- extended = 0;
- - fread(&temp,2,1,fi);
- + err = fread(&temp,2,1,fi);
- + assert (err == 1);
- fs = temp;
- }
- @@ -146,20 +155,25 @@
- printf("Can't open file %s\n", filename);
- exit(0);
- }
- - fread(&fnc,2,1,fo);
- + err = fread(&fnc,2,1,fo);
- + assert (err == 1);
- +
- if (fnc == 0xFFFF) {
- if (extended == 0) {
- printf("Wrong header (u7) in object\n");
- exit(0);
- }
- - fread(&fnc,2,1,fo);
- - fread(&fsc,4,1,fo);
- + err = fread(&fnc,2,1,fo);
- + assert (err == 1);
- + err = fread(&fsc,4,1,fo);
- + assert (err == 1);
- } else {
- if (extended == 1) {
- printf("Wrong header (extended) in object\n");
- exit(0);
- }
- - fread(&temp,2,1,fo);
- + err = fread(&temp,2,1,fo);
- + assert (err == 1);
- fsc = temp;
- }
- if (fnc!=fn)
- Index: files/zip/zip.c
- ===================================================================
- --- files/zip/zip.c (revision 7169)
- +++ files/zip/zip.c (working copy)
- @@ -168,13 +168,15 @@
- ll->first_block = ll->last_block = NULL;
- }
- +/* Currently not being used */
- +#if 0
- static void free_linkedlist(linkedlist_data* ll)
- {
- free_datablock(ll->first_block);
- ll->first_block = ll->last_block = NULL;
- }
- +#endif
- -
- static int add_data_in_datablock(linkedlist_data* ll, const void* buf, uLong len)
- {
- linkedlist_datablock_internal* ldi;
- @@ -227,6 +229,8 @@
- }
- +/* Currently not being used */
- +#if 0
- static int write_datablock(FILE* fout, linkedlist_data* ll)
- {
- linkedlist_datablock_internal* ldi;
- @@ -240,6 +244,7 @@
- }
- return ZIP_OK;
- }
- +#endif
- /****************************************************************************/
- Index: audio/midi_drivers/timidity/timidity_readmidi.cpp
- ===================================================================
- --- audio/midi_drivers/timidity/timidity_readmidi.cpp (revision 7169)
- +++ audio/midi_drivers/timidity/timidity_readmidi.cpp (working copy)
- @@ -79,9 +79,11 @@
- {
- sint32 l=0;
- uint8 c;
- + size_t err;
- for (;;)
- {
- - fread(&c,1,1,fp);
- + err = fread(&c,1,1,fp);
- + assert (err == 1);
- l += (c & 0x7f);
- if (!(c & 0x80)) return l;
- l<<=7;
- @@ -126,6 +128,7 @@
- uint8 me, type, a,b,c;
- sint32 len;
- MidiEventList *event;
- + size_t err;
- for (;;)
- {
- @@ -144,7 +147,9 @@
- }
- else if(me==0xFF) /* Meta event */
- {
- - fread(&type,1,1,fp);
- + err = fread(&type,1,1,fp);
- + assert (err == 1);
- +
- len=getvl();
- if (type>0 && type<16)
- {
- @@ -160,7 +165,12 @@
- return MAGIC_EOT;
- case 0x51: /* Tempo */
- - fread(&a,1,1,fp); fread(&b,1,1,fp); fread(&c,1,1,fp);
- + err = fread(&a,1,1,fp);
- + assert (err == 1);
- + err = fread(&b,1,1,fp);
- + assert (err == 1);
- + err = fread(&c,1,1,fp);
- + assert (err == 1);
- MIDIEVENT(at, ME_TEMPO, c, a, b);
- default:
- @@ -177,28 +187,33 @@
- {
- lastchan=a & 0x0F;
- laststatus=(a>>4) & 0x07;
- - fread(&a, 1,1, fp);
- + err = fread(&a, 1,1, fp);
- + assert (err == 1);
- a &= 0x7F;
- }
- switch(laststatus)
- {
- case 0: /* Note off */
- - fread(&b, 1,1, fp);
- + err = fread(&b, 1,1, fp);
- + assert (err == 1);
- b &= 0x7F;
- MIDIEVENT(at, ME_NOTEOFF, lastchan, a,b);
- case 1: /* Note on */
- - fread(&b, 1,1, fp);
- + err = fread(&b, 1,1, fp);
- + assert (err == 1);
- b &= 0x7F;
- MIDIEVENT(at, ME_NOTEON, lastchan, a,b);
- case 2: /* Key Pressure */
- - fread(&b, 1,1, fp);
- + err = fread(&b, 1,1, fp);
- + assert (err == 1);
- b &= 0x7F;
- MIDIEVENT(at, ME_KEYPRESSURE, lastchan, a, b);
- case 3: /* Control change */
- - fread(&b, 1,1, fp);
- + err = fread(&b, 1,1, fp);
- + assert (err == 1);
- b &= 0x7F;
- {
- int control=255;
- @@ -281,7 +296,8 @@
- break;
- case 6: /* Pitch wheel */
- - fread(&b, 1,1, fp);
- + err = fread(&b, 1,1, fp);
- + assert (err == 1);
- b &= 0x7F;
- MIDIEVENT(at, ME_PITCHWHEEL, lastchan, a, b);
- @@ -539,6 +555,7 @@
- sint16 format, tracks, divisions_tmp;
- int i;
- char tmp[4];
- + size_t err;
- fp=mfp;
- event_count=0;
- @@ -565,9 +582,12 @@
- return 0;
- }
- - fread(&format, 2, 1, fp);
- - fread(&tracks, 2, 1, fp);
- - fread(&divisions_tmp, 2, 1, fp);
- + err = fread(&format, 2, 1, fp);
- + assert (err == 1);
- + err = fread(&tracks, 2, 1, fp);
- + assert (err == 1);
- + err = fread(&divisions_tmp, 2, 1, fp);
- + assert (err == 1);
- format=BE_SHORT(format);
- tracks=BE_SHORT(tracks);
- divisions_tmp=BE_SHORT(divisions_tmp);
- Index: audio/midi_drivers/UnixSeqMidiDriver.cpp
- ===================================================================
- --- audio/midi_drivers/UnixSeqMidiDriver.cpp (revision 7169)
- +++ audio/midi_drivers/UnixSeqMidiDriver.cpp (working copy)
- @@ -70,6 +70,7 @@
- void UnixSeqMidiDriver::send(uint32 b) {
- unsigned char buf[256];
- int position = 0;
- + size_t err;
- switch (b & 0xF0) {
- case 0x80:
- @@ -107,7 +108,8 @@
- break;
- }
- - ::write(device, buf, position);
- + err = ::write(device, buf, position);
- + assert (err == position);
- }
- void UnixSeqMidiDriver::send_sysex(uint8 status,const uint8 *msg,uint16 length)
- @@ -121,6 +123,7 @@
- unsigned char buf [2048];
- int position = 0;
- const uint8 *chr = msg;
- + size_t err;
- buf[position++] = SEQ_MIDIPUTC;
- buf[position++] = status;
- @@ -134,7 +137,8 @@
- buf[position++] = 0;
- }
- - ::write (device, buf, position);
- + err = ::write (device, buf, position);
- + assert (err == position);
- }
- #endif
- Index: gumps/Newfile_gump.cc
- ===================================================================
- --- gumps/Newfile_gump.cc (revision 7169)
- +++ gumps/Newfile_gump.cc (working copy)
- @@ -595,7 +595,7 @@
- else if ((details->real_day%10) == 3 && details->real_day != 13)
- suffix = "rd";
- - snprintf (info, 320, infostring, party[0].name,
- + snprintf (info, sizeof (info), infostring, party[0].name,
- party[0].exp, party[0].health,
- party[0].str, party[0].dext,
- party[0].intel, party[0].training,
- @@ -606,7 +606,7 @@
- if (filename)
- {
- - std::strncat (info, "\nFile: ", 320);
- + std::strncat (info, "\nFile: ", sizeof (info) - strlen (info));
- int offset = strlen(filename);
- @@ -618,7 +618,7 @@
- break;
- }
- }
- - std::strncat (info, filename+offset, 320);
- + std::strncat (info, filename+offset, sizeof (info) - strlen (info));
- }
- @@ -631,7 +631,7 @@
- {
- char info[64] = {0};
- - std::strncat (info, "File: ", 64);
- + std::strncat (info, "File: ", sizeof (info));
- int offset = strlen(filename);
- @@ -643,7 +643,7 @@
- break;
- }
- }
- - std::strncat (info, filename+offset, 64);
- + std::strncat (info, filename+offset, sizeof (info) - strlen (info));
- sman->paint_text_box (4, info, x+infox, y+infoy, infow, infoh);
- }
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement