Skip to content
Snippets Groups Projects
Commit d88ebed3 authored by Rob Swindell's avatar Rob Swindell :speech_balloon:
Browse files

The great 'long int' purge of 2023 part 4

[s]printf format fix-ups
parent ae02043d
No related branches found
No related tags found
1 merge request!463MRC mods by Codefenix (2024-10-20)
Pipeline #3816 failed
...@@ -440,7 +440,7 @@ int main(int argc, char **argv) ...@@ -440,7 +440,7 @@ int main(int argc, char **argv)
fprintf(stderr,"%sHeader length mismatch\n",beep); fprintf(stderr,"%sHeader length mismatch\n",beep);
msgerr=TRUE; msgerr=TRUE;
if(extinfo) if(extinfo)
printf("MSGERR: Header length (%hu) does not match calculcated length (%lu)\n" printf("MSGERR: Header length (%hu) does not match calculcated length (%u)\n"
,msg.hdr.length,smb_getmsghdrlen(&msg)); ,msg.hdr.length,smb_getmsghdrlen(&msg));
hdrlenerr++; hdrlenerr++;
} }
......
...@@ -1520,7 +1520,7 @@ static void pop3_thread(void* arg) ...@@ -1520,7 +1520,7 @@ static void pop3_thread(void* arg)
continue; continue;
} }
if(!strnicmp(buf, "LIST",4)) { if(!strnicmp(buf, "LIST",4)) {
sockprintf(socket,client.protocol,session,"+OK %" PRIu32 " %lu",msgnum,smb_getmsgtxtlen(&msg)); sockprintf(socket,client.protocol,session,"+OK %" PRIu32 " %u",msgnum,smb_getmsgtxtlen(&msg));
} else /* UIDL */ } else /* UIDL */
sockprintf(socket,client.protocol,session,"+OK %" PRIu32 " %u",msgnum,msg.hdr.number); sockprintf(socket,client.protocol,session,"+OK %" PRIu32 " %u",msgnum,msg.hdr.number);
...@@ -1552,7 +1552,7 @@ static void pop3_thread(void* arg) ...@@ -1552,7 +1552,7 @@ static void pop3_thread(void* arg)
break; break;
} }
if(!strnicmp(buf, "LIST",4)) { if(!strnicmp(buf, "LIST",4)) {
sockprintf(socket,client.protocol,session,"%u %lu",l+1,smb_getmsgtxtlen(&msg)); sockprintf(socket,client.protocol,session,"%u %u",l+1,smb_getmsgtxtlen(&msg));
} else /* UIDL */ } else /* UIDL */
sockprintf(socket,client.protocol,session,"%u %u",l+1,msg.hdr.number); sockprintf(socket,client.protocol,session,"%u %u",l+1,msg.hdr.number);
......
...@@ -76,7 +76,7 @@ int smb_addmsg(smb_t* smb, smbmsg_t* msg, int storage, int dupechk_hashes ...@@ -76,7 +76,7 @@ int smb_addmsg(smb_t* smb, smbmsg_t* msg, int storage, int dupechk_hashes
if(smb_findhash(smb, hashes, &found, dupechk_hashes, /* mark? */FALSE)==SMB_SUCCESS) { if(smb_findhash(smb, hashes, &found, dupechk_hashes, /* mark? */FALSE)==SMB_SUCCESS) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s duplicate %s: %s found in message #%lu" ,"%s duplicate %s: %s found in message #%u"
,__FUNCTION__ ,__FUNCTION__
,smb_hashsourcetype(found.source) ,smb_hashsourcetype(found.source)
,smb_hashsource(msg,found.source) ,smb_hashsource(msg,found.source)
...@@ -188,7 +188,7 @@ int smb_addmsg(smb_t* smb, smbmsg_t* msg, int storage, int dupechk_hashes ...@@ -188,7 +188,7 @@ int smb_addmsg(smb_t* smb, smbmsg_t* msg, int storage, int dupechk_hashes
if(smb_fwrite(smb,body,bodylen,smb->sdt_fp)!=bodylen) { if(smb_fwrite(smb,body,bodylen,smb->sdt_fp)!=bodylen) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s writing body (%ld bytes)" ,"%s writing body (%d bytes)"
,__FUNCTION__ ,__FUNCTION__
,(int)bodylen); ,(int)bodylen);
retval=SMB_ERR_WRITE; retval=SMB_ERR_WRITE;
...@@ -211,7 +211,7 @@ int smb_addmsg(smb_t* smb, smbmsg_t* msg, int storage, int dupechk_hashes ...@@ -211,7 +211,7 @@ int smb_addmsg(smb_t* smb, smbmsg_t* msg, int storage, int dupechk_hashes
if(smb_fwrite(smb,tail,taillen-sizeof(xlat),smb->sdt_fp)!=taillen-sizeof(xlat)) { if(smb_fwrite(smb,tail,taillen-sizeof(xlat),smb->sdt_fp)!=taillen-sizeof(xlat)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s writing tail (%ld bytes)" ,"%s writing tail (%d bytes)"
,__FUNCTION__ ,__FUNCTION__
,(int)(taillen-sizeof(xlat))); ,(int)(taillen-sizeof(xlat)));
retval=SMB_ERR_WRITE; retval=SMB_ERR_WRITE;
......
...@@ -239,7 +239,7 @@ int smb_incdat(smb_t* smb, off_t offset, uint length, uint16_t refs) ...@@ -239,7 +239,7 @@ int smb_incdat(smb_t* smb, off_t offset, uint length, uint16_t refs)
} }
i+=refs; i+=refs;
if(fseek(smb->sda_fp,-(int)sizeof(i),SEEK_CUR)) { if(fseek(smb->sda_fp,-(int)sizeof(i),SEEK_CUR)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error),"%s rewinding %ld", __FUNCTION__, -(int)sizeof(i)); safe_snprintf(smb->last_error,sizeof(smb->last_error),"%s rewinding %d", __FUNCTION__, -(int)sizeof(i));
return(SMB_ERR_SEEK); return(SMB_ERR_SEEK);
} }
if(!fwrite(&i,sizeof(i),1,smb->sda_fp)) { if(!fwrite(&i,sizeof(i),1,smb->sda_fp)) {
...@@ -318,7 +318,7 @@ int smb_freemsghdr(smb_t* smb, off_t offset, uint length) ...@@ -318,7 +318,7 @@ int smb_freemsghdr(smb_t* smb, off_t offset, uint length)
} }
if(fseeko(smb->sha_fp, sha_offset, SEEK_SET)) { if(fseeko(smb->sha_fp, sha_offset, SEEK_SET)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error),"%s seeking to %ld", __FUNCTION__, (int)sha_offset); safe_snprintf(smb->last_error,sizeof(smb->last_error),"%s seeking to %d", __FUNCTION__, (int)sha_offset);
return(SMB_ERR_SEEK); return(SMB_ERR_SEEK);
} }
for(l=0;l<blocks;l++) for(l=0;l<blocks;l++)
...@@ -400,7 +400,7 @@ off_t smb_allochdr(smb_t* smb, uint length) ...@@ -400,7 +400,7 @@ off_t smb_allochdr(smb_t* smb, uint length)
} }
clearerr(smb->sha_fp); clearerr(smb->sha_fp);
if(fseek(smb->sha_fp,offset/SHD_BLOCK_LEN,SEEK_SET)) { if(fseek(smb->sha_fp,offset/SHD_BLOCK_LEN,SEEK_SET)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error),"%s seeking to %ld", __FUNCTION__, offset/SHD_BLOCK_LEN); safe_snprintf(smb->last_error,sizeof(smb->last_error),"%s seeking to %d", __FUNCTION__, offset/SHD_BLOCK_LEN);
return(SMB_ERR_SEEK); return(SMB_ERR_SEEK);
} }
for(l=0;l<blocks;l++) for(l=0;l<blocks;l++)
......
...@@ -303,7 +303,7 @@ int smb_hashmsg(smb_t* smb, smbmsg_t* msg, const uchar* text, BOOL update) ...@@ -303,7 +303,7 @@ int smb_hashmsg(smb_t* smb, smbmsg_t* msg, const uchar* text, BOOL update)
if(smb_findhash(smb, hashes, &found, SMB_HASH_SOURCE_DUPE, update)==SMB_SUCCESS && !update) { if(smb_findhash(smb, hashes, &found, SMB_HASH_SOURCE_DUPE, update)==SMB_SUCCESS && !update) {
retval=SMB_DUPE_MSG; retval=SMB_DUPE_MSG;
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s duplicate %s: %s found in message #%lu", __FUNCTION__ ,"%s duplicate %s: %s found in message #%u", __FUNCTION__
,smb_hashsourcetype(found.source) ,smb_hashsourcetype(found.source)
,smb_hashsource(msg,found.source) ,smb_hashsource(msg,found.source)
,(uint)found.number); ,(uint)found.number);
......
...@@ -251,7 +251,7 @@ int smb_trunchdr(smb_t* smb) ...@@ -251,7 +251,7 @@ int smb_trunchdr(smb_t* smb)
else else
if(time(NULL)-start>=(time_t)smb->retry_time) { /* Time-out */ if(time(NULL)-start>=(time_t)smb->retry_time) { /* Time-out */
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s timeout changing header file size (retry_time=%lu)", __FUNCTION__ ,"%s timeout changing header file size (retry_time=%u)", __FUNCTION__
,(uint)smb->retry_time); ,(uint)smb->retry_time);
return(SMB_ERR_TIMEOUT); return(SMB_ERR_TIMEOUT);
} }
...@@ -385,7 +385,7 @@ BOOL smb_valid_hdr_offset(smb_t* smb, uint offset) ...@@ -385,7 +385,7 @@ BOOL smb_valid_hdr_offset(smb_t* smb, uint offset)
if(offset<sizeof(smbhdr_t)+sizeof(smbstatus_t) if(offset<sizeof(smbhdr_t)+sizeof(smbstatus_t)
|| offset<smb->status.header_offset) { || offset<smb->status.header_offset) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s invalid header offset: %lu (0x%lX)", __FUNCTION__ ,"%s invalid header offset: %u (0x%X)", __FUNCTION__
,offset,offset); ,offset,offset);
return(FALSE); return(FALSE);
} }
...@@ -464,14 +464,14 @@ int smb_getmsgidx(smb_t* smb, smbmsg_t* msg) ...@@ -464,14 +464,14 @@ int smb_getmsgidx(smb_t* smb, smbmsg_t* msg)
byte_offset=msg->idx_offset*idxreclen; byte_offset=msg->idx_offset*idxreclen;
if(byte_offset>=length) { if(byte_offset>=length) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s invalid index offset: %ld, byte offset: %ld, length: %" PRIdOFF, __FUNCTION__ ,"%s invalid index offset: %d, byte offset: %d, length: %" PRIdOFF, __FUNCTION__
,(int)msg->idx_offset, byte_offset, length); ,(int)msg->idx_offset, byte_offset, length);
return(SMB_ERR_HDR_OFFSET); return(SMB_ERR_HDR_OFFSET);
} }
if(ftell(smb->sid_fp) != byte_offset) { if(ftell(smb->sid_fp) != byte_offset) {
if(fseek(smb->sid_fp,byte_offset,SEEK_SET)) { if(fseek(smb->sid_fp,byte_offset,SEEK_SET)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s %d '%s' seeking to offset %ld (byte %lu) in index file", __FUNCTION__ ,"%s %d '%s' seeking to offset %d (byte %u) in index file", __FUNCTION__
,get_errno(),STRERROR(get_errno()) ,get_errno(),STRERROR(get_errno())
,(int)msg->idx_offset,byte_offset); ,(int)msg->idx_offset,byte_offset);
return(SMB_ERR_SEEK); return(SMB_ERR_SEEK);
...@@ -479,7 +479,7 @@ int smb_getmsgidx(smb_t* smb, smbmsg_t* msg) ...@@ -479,7 +479,7 @@ int smb_getmsgidx(smb_t* smb, smbmsg_t* msg)
} }
if(smb_fread(smb,&msg->idx,sizeof(msg->idx),smb->sid_fp)!=sizeof(msg->idx)) { if(smb_fread(smb,&msg->idx,sizeof(msg->idx),smb->sid_fp)!=sizeof(msg->idx)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s reading index at offset %ld (byte %lu)", __FUNCTION__ ,"%s reading index at offset %d (byte %u)", __FUNCTION__
,(int)msg->idx_offset,byte_offset); ,(int)msg->idx_offset,byte_offset);
return(SMB_ERR_READ); return(SMB_ERR_READ);
} }
...@@ -493,25 +493,25 @@ int smb_getmsgidx(smb_t* smb, smbmsg_t* msg) ...@@ -493,25 +493,25 @@ int smb_getmsgidx(smb_t* smb, smbmsg_t* msg)
l=total/2; /* Start at middle index */ l=total/2; /* Start at middle index */
while(1) { while(1) {
if(l>=total) { if(l>=total) {
safe_snprintf(smb->last_error,sizeof(smb->last_error),"%s msg %lu not found" safe_snprintf(smb->last_error,sizeof(smb->last_error),"%s msg %u not found"
, __FUNCTION__, (uint)msg->hdr.number); , __FUNCTION__, (uint)msg->hdr.number);
return(SMB_ERR_NOT_FOUND); return(SMB_ERR_NOT_FOUND);
} }
if(fseek(smb->sid_fp,l*idxreclen,SEEK_SET)) { if(fseek(smb->sid_fp,l*idxreclen,SEEK_SET)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s %d '%s' seeking to offset %lu (byte %lu) in index file", __FUNCTION__ ,"%s %d '%s' seeking to offset %u (byte %u) in index file", __FUNCTION__
,get_errno(),STRERROR(get_errno()) ,get_errno(),STRERROR(get_errno())
,l,l*idxreclen); ,l,(uint)(l*idxreclen));
return(SMB_ERR_SEEK); return(SMB_ERR_SEEK);
} }
if(smb_fread(smb,&idx,sizeof(idx),smb->sid_fp)!=sizeof(idx)) { if(smb_fread(smb,&idx,sizeof(idx),smb->sid_fp)!=sizeof(idx)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s reading index at offset %lu (byte %lu)", __FUNCTION__ ,"%s reading index at offset %u (byte %u)", __FUNCTION__
,l,l*sizeof(idx)); ,l,(uint)(l*sizeof(idx)));
return(SMB_ERR_READ); return(SMB_ERR_READ);
} }
if(bot==top-1 && idx.number!=msg->hdr.number) { if(bot==top-1 && idx.number!=msg->hdr.number) {
safe_snprintf(smb->last_error,sizeof(smb->last_error),"%s msg %lu not found" safe_snprintf(smb->last_error,sizeof(smb->last_error),"%s msg %u not found"
, __FUNCTION__, (uint)msg->hdr.number); , __FUNCTION__, (uint)msg->hdr.number);
return(SMB_ERR_NOT_FOUND); return(SMB_ERR_NOT_FOUND);
} }
...@@ -988,7 +988,7 @@ int smb_getmsghdr(smb_t* smb, smbmsg_t* msg) ...@@ -988,7 +988,7 @@ int smb_getmsghdr(smb_t* smb, smbmsg_t* msg)
if(offset != msg->idx.offset) { if(offset != msg->idx.offset) {
if(fseek(smb->shd_fp,msg->idx.offset,SEEK_SET) != 0) { if(fseek(smb->shd_fp,msg->idx.offset,SEEK_SET) != 0) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s %d '%s' seeking to offset %lu in header file", __FUNCTION__ ,"%s %d '%s' seeking to offset %u in header file", __FUNCTION__
,get_errno(),STRERROR(get_errno()) ,get_errno(),STRERROR(get_errno())
,(uint)msg->idx.offset); ,(uint)msg->idx.offset);
return(SMB_ERR_SEEK); return(SMB_ERR_SEEK);
...@@ -1002,12 +1002,12 @@ int smb_getmsghdr(smb_t* smb, smbmsg_t* msg) ...@@ -1002,12 +1002,12 @@ int smb_getmsghdr(smb_t* smb, smbmsg_t* msg)
msg->idx_offset=offset; msg->idx_offset=offset;
if(smb_fread(smb,&msg->hdr,sizeof(msghdr_t),smb->shd_fp)!=sizeof(msghdr_t)) { if(smb_fread(smb,&msg->hdr,sizeof(msghdr_t),smb->shd_fp)!=sizeof(msghdr_t)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s reading msg header at offset %lu", __FUNCTION__, (uint)msg->idx.offset); ,"%s reading msg header at offset %u", __FUNCTION__, (uint)msg->idx.offset);
return(SMB_ERR_READ); return(SMB_ERR_READ);
} }
if(memcmp(msg->hdr.msghdr_id,SHD_HEADER_ID,LEN_HEADER_ID)) { if(memcmp(msg->hdr.msghdr_id,SHD_HEADER_ID,LEN_HEADER_ID)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s corrupt message header ID (%02X %02X %02X %02X) at offset %lu", __FUNCTION__ ,"%s corrupt message header ID (%02X %02X %02X %02X) at offset %u", __FUNCTION__
,msg->hdr.msghdr_id[0] ,msg->hdr.msghdr_id[0]
,msg->hdr.msghdr_id[1] ,msg->hdr.msghdr_id[1]
,msg->hdr.msghdr_id[2] ,msg->hdr.msghdr_id[2]
...@@ -1017,7 +1017,7 @@ int smb_getmsghdr(smb_t* smb, smbmsg_t* msg) ...@@ -1017,7 +1017,7 @@ int smb_getmsghdr(smb_t* smb, smbmsg_t* msg)
} }
if(msg->hdr.version<0x110) { if(msg->hdr.version<0x110) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s insufficient header version: %X at offset %lu", __FUNCTION__ ,"%s insufficient header version: %X at offset %u", __FUNCTION__
,msg->hdr.version, (uint)msg->idx.offset); ,msg->hdr.version, (uint)msg->idx.offset);
return(SMB_ERR_HDR_VER); return(SMB_ERR_HDR_VER);
} }
...@@ -1026,7 +1026,7 @@ int smb_getmsghdr(smb_t* smb, smbmsg_t* msg) ...@@ -1026,7 +1026,7 @@ int smb_getmsghdr(smb_t* smb, smbmsg_t* msg)
&& (msg->dfield = malloc(sizeof(*msg->dfield)*msg->hdr.total_dfields)) == NULL) { && (msg->dfield = malloc(sizeof(*msg->dfield)*msg->hdr.total_dfields)) == NULL) {
smb_freemsgmem(msg); smb_freemsgmem(msg);
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s malloc failure of %ld bytes for %u data fields", __FUNCTION__ ,"%s malloc failure of %d bytes for %u data fields", __FUNCTION__
,(int)(sizeof(*msg->dfield) * msg->hdr.total_dfields), msg->hdr.total_dfields); ,(int)(sizeof(*msg->dfield) * msg->hdr.total_dfields), msg->hdr.total_dfields);
return(SMB_ERR_MEM); return(SMB_ERR_MEM);
} }
...@@ -1165,7 +1165,7 @@ int smb_copymsgmem(smb_t* smb, smbmsg_t* msg, smbmsg_t* srcmsg) ...@@ -1165,7 +1165,7 @@ int smb_copymsgmem(smb_t* smb, smbmsg_t* msg, smbmsg_t* srcmsg)
if((msg->dfield=(dfield_t *)malloc(msg->hdr.total_dfields*sizeof(dfield_t)))==NULL) { if((msg->dfield=(dfield_t *)malloc(msg->hdr.total_dfields*sizeof(dfield_t)))==NULL) {
if(smb!=NULL) if(smb!=NULL)
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s malloc failure of %ld bytes for %d data fields", __FUNCTION__ ,"%s malloc failure of %d bytes for %d data fields", __FUNCTION__
,(int)(msg->hdr.total_dfields*sizeof(dfield_t)), msg->hdr.total_dfields); ,(int)(msg->hdr.total_dfields*sizeof(dfield_t)), msg->hdr.total_dfields);
return(SMB_ERR_MEM); return(SMB_ERR_MEM);
} }
...@@ -1177,7 +1177,7 @@ int smb_copymsgmem(smb_t* smb, smbmsg_t* msg, smbmsg_t* srcmsg) ...@@ -1177,7 +1177,7 @@ int smb_copymsgmem(smb_t* smb, smbmsg_t* msg, smbmsg_t* srcmsg)
if((msg->hfield=(hfield_t *)malloc(msg->total_hfields*sizeof(hfield_t)))==NULL) { if((msg->hfield=(hfield_t *)malloc(msg->total_hfields*sizeof(hfield_t)))==NULL) {
if(smb!=NULL) if(smb!=NULL)
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s malloc failure of %ld bytes for %d header fields", __FUNCTION__ ,"%s malloc failure of %d bytes for %d header fields", __FUNCTION__
,(int)(msg->total_hfields*sizeof(hfield_t)), msg->total_hfields); ,(int)(msg->total_hfields*sizeof(hfield_t)), msg->total_hfields);
return(SMB_ERR_MEM); return(SMB_ERR_MEM);
} }
...@@ -1187,7 +1187,7 @@ int smb_copymsgmem(smb_t* smb, smbmsg_t* msg, smbmsg_t* srcmsg) ...@@ -1187,7 +1187,7 @@ int smb_copymsgmem(smb_t* smb, smbmsg_t* msg, smbmsg_t* srcmsg)
if((msg->hfield_dat=(void**)malloc(msg->total_hfields*sizeof(void*)))==NULL) { if((msg->hfield_dat=(void**)malloc(msg->total_hfields*sizeof(void*)))==NULL) {
if(smb!=NULL) if(smb!=NULL)
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s malloc failure of %ld bytes for %d header fields", __FUNCTION__ ,"%s malloc failure of %d bytes for %d header fields", __FUNCTION__
,(int)(msg->total_hfields*sizeof(void*)), msg->total_hfields); ,(int)(msg->total_hfields*sizeof(void*)), msg->total_hfields);
return(SMB_ERR_MEM); return(SMB_ERR_MEM);
} }
...@@ -1520,7 +1520,7 @@ int smb_addcrc(smb_t* smb, uint32_t crc) ...@@ -1520,7 +1520,7 @@ int smb_addcrc(smb_t* smb, uint32_t crc)
else else
if(time(NULL)-start>=(time_t)smb->retry_time) { if(time(NULL)-start>=(time_t)smb->retry_time) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s timeout opening %s (retry_time=%lu)", __FUNCTION__ ,"%s timeout opening %s (retry_time=%u)", __FUNCTION__
,str, (uint)smb->retry_time); ,str, (uint)smb->retry_time);
return(SMB_ERR_TIMEOUT); return(SMB_ERR_TIMEOUT);
} }
...@@ -1581,7 +1581,7 @@ int smb_addcrc(smb_t* smb, uint32_t crc) ...@@ -1581,7 +1581,7 @@ int smb_addcrc(smb_t* smb, uint32_t crc)
if(wr!=sizeof(crc)) { if(wr!=sizeof(crc)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s %d '%s' writing %ld bytes", __FUNCTION__ ,"%s %d '%s' writing %d bytes", __FUNCTION__
,get_errno(),STRERROR(get_errno()),(int)sizeof(crc)); ,get_errno(),STRERROR(get_errno()),(int)sizeof(crc));
return(SMB_ERR_WRITE); return(SMB_ERR_WRITE);
} }
...@@ -1616,7 +1616,7 @@ int smb_addmsghdr(smb_t* smb, smbmsg_t* msg, int storage) ...@@ -1616,7 +1616,7 @@ int smb_addmsghdr(smb_t* smb, smbmsg_t* msg, int storage)
if(hdrlen>SMB_MAX_HDR_LEN) { /* headers are limited to 64k in size */ if(hdrlen>SMB_MAX_HDR_LEN) { /* headers are limited to 64k in size */
smb_unlocksmbhdr(smb); smb_unlocksmbhdr(smb);
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s illegal message header length (%lu > %u)", __FUNCTION__ ,"%s illegal message header length (%u > %u)", __FUNCTION__
,hdrlen,SMB_MAX_HDR_LEN); ,hdrlen,SMB_MAX_HDR_LEN);
return(SMB_ERR_HDR_LEN); return(SMB_ERR_HDR_LEN);
} }
...@@ -1629,8 +1629,8 @@ int smb_addmsghdr(smb_t* smb, smbmsg_t* msg, int storage) ...@@ -1629,8 +1629,8 @@ int smb_addmsghdr(smb_t* smb, smbmsg_t* msg, int storage)
idxlen = filelength(fileno(smb->sid_fp)); idxlen = filelength(fileno(smb->sid_fp));
if(idxlen != (smb->status.total_msgs * idxreclen)) { if(idxlen != (smb->status.total_msgs * idxreclen)) {
safe_snprintf(smb->last_error, sizeof(smb->last_error) safe_snprintf(smb->last_error, sizeof(smb->last_error)
,"%s index file length (%" PRIdOFF "), expected (%ld)", __FUNCTION__ ,"%s index file length (%" PRIdOFF "), expected (%d)", __FUNCTION__
,idxlen, smb->status.total_msgs * idxreclen); ,idxlen, (uint)(smb->status.total_msgs * idxreclen));
smb_unlocksmbhdr(smb); smb_unlocksmbhdr(smb);
return SMB_ERR_FILE_LEN; return SMB_ERR_FILE_LEN;
} }
...@@ -1851,8 +1851,8 @@ int smb_putmsgidx(smb_t* smb, smbmsg_t* msg) ...@@ -1851,8 +1851,8 @@ int smb_putmsgidx(smb_t* smb, smbmsg_t* msg)
length = filelength(fileno(smb->sid_fp)); length = filelength(fileno(smb->sid_fp));
if(length < (int)(msg->idx_offset*idxreclen)) { if(length < (int)(msg->idx_offset*idxreclen)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s invalid index offset: %ld, byte offset: %lu, length: %" PRIdOFF, __FUNCTION__ ,"%s invalid index offset: %d, byte offset: %u, length: %" PRIdOFF, __FUNCTION__
,(int)msg->idx_offset, msg->idx_offset*idxreclen, length); ,(int)msg->idx_offset, (uint)(msg->idx_offset*idxreclen), length);
return(SMB_ERR_HDR_OFFSET); return(SMB_ERR_HDR_OFFSET);
} }
if(fseek(smb->sid_fp,msg->idx_offset*idxreclen,SEEK_SET)) { if(fseek(smb->sid_fp,msg->idx_offset*idxreclen,SEEK_SET)) {
...@@ -1902,7 +1902,7 @@ int smb_putmsghdr(smb_t* smb, smbmsg_t* msg) ...@@ -1902,7 +1902,7 @@ int smb_putmsghdr(smb_t* smb, smbmsg_t* msg)
clearerr(smb->shd_fp); clearerr(smb->shd_fp);
if(fseek(smb->shd_fp,msg->idx.offset,SEEK_SET)) { if(fseek(smb->shd_fp,msg->idx.offset,SEEK_SET)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s %d '%s' seeking to %lu in header file", __FUNCTION__ ,"%s %d '%s' seeking to %u in header file", __FUNCTION__
,get_errno(),STRERROR(get_errno()), (uint)msg->idx.offset); ,get_errno(),STRERROR(get_errno()), (uint)msg->idx.offset);
return(SMB_ERR_SEEK); return(SMB_ERR_SEEK);
} }
...@@ -1911,13 +1911,13 @@ int smb_putmsghdr(smb_t* smb, smbmsg_t* msg) ...@@ -1911,13 +1911,13 @@ int smb_putmsghdr(smb_t* smb, smbmsg_t* msg)
hdrlen=smb_getmsghdrlen(msg); hdrlen=smb_getmsghdrlen(msg);
if(hdrlen>SMB_MAX_HDR_LEN) { /* headers are limited to 64k in size */ if(hdrlen>SMB_MAX_HDR_LEN) { /* headers are limited to 64k in size */
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s illegal message header length (%lu > %u)", __FUNCTION__ ,"%s illegal message header length (%u > %u)", __FUNCTION__
,hdrlen,SMB_MAX_HDR_LEN); ,hdrlen,SMB_MAX_HDR_LEN);
return(SMB_ERR_HDR_LEN); return(SMB_ERR_HDR_LEN);
} }
if(smb_hdrblocks(hdrlen) > smb_hdrblocks(msg->hdr.length)) { if(smb_hdrblocks(hdrlen) > smb_hdrblocks(msg->hdr.length)) {
safe_snprintf(smb->last_error,sizeof(smb->last_error) safe_snprintf(smb->last_error,sizeof(smb->last_error)
,"%s illegal header length increase: %lu (%lu blocks, %hu hfields, %hu dfields) vs %hu (%lu blocks)", __FUNCTION__ ,"%s illegal header length increase: %u (%u blocks, %hu hfields, %hu dfields) vs %hu (%u blocks)", __FUNCTION__
,hdrlen, smb_hdrblocks(hdrlen), msg->total_hfields, msg->hdr.total_dfields ,hdrlen, smb_hdrblocks(hdrlen), msg->total_hfields, msg->hdr.total_dfields
,msg->hdr.length, smb_hdrblocks(msg->hdr.length)); ,msg->hdr.length, smb_hdrblocks(msg->hdr.length));
return(SMB_ERR_HDR_LEN); return(SMB_ERR_HDR_LEN);
...@@ -2030,7 +2030,7 @@ int smb_create(smb_t* smb) ...@@ -2030,7 +2030,7 @@ int smb_create(smb_t* smb)
SAFEPRINTF(str,"%s.ini",smb->file); SAFEPRINTF(str,"%s.ini",smb->file);
if((fp = fopen(str, "w")) != NULL) { if((fp = fopen(str, "w")) != NULL) {
fprintf(fp, "Created = 0x%lx\n", (int)time(NULL)); fprintf(fp, "Created = 0x%x\n", (int)time(NULL));
fclose(fp); fclose(fp);
} }
SAFEPRINTF(str,"%s.sda",smb->file); SAFEPRINTF(str,"%s.sda",smb->file);
......
...@@ -56,7 +56,7 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode) ...@@ -56,7 +56,7 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode)
length=strlen(str)+2; /* +2 for crlf */ length=strlen(str)+2; /* +2 for crlf */
if((p=(char*)realloc(buf,l+length+1))==NULL) { if((p=(char*)realloc(buf,l+length+1))==NULL) {
safe_snprintf(smb->last_error, sizeof(smb->last_error) safe_snprintf(smb->last_error, sizeof(smb->last_error)
,"%s realloc failure of %ld bytes for comment buffer" ,"%s realloc failure of %d bytes for comment buffer"
, __FUNCTION__, l+length+1); , __FUNCTION__, l+length+1);
free(buf); free(buf);
return(NULL); return(NULL);
...@@ -67,7 +67,7 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode) ...@@ -67,7 +67,7 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode)
if(l) { /* Add a blank line after comments */ if(l) { /* Add a blank line after comments */
if((p=(char*)realloc(buf,l+3))==NULL) { if((p=(char*)realloc(buf,l+3))==NULL) {
safe_snprintf(smb->last_error, sizeof(smb->last_error) safe_snprintf(smb->last_error, sizeof(smb->last_error)
,"%s realloc failure of %ld bytes for comment buffer" ,"%s realloc failure of %d bytes for comment buffer"
, __FUNCTION__, l+3); , __FUNCTION__, l+3);
free(buf); free(buf);
return(NULL); return(NULL);
...@@ -83,7 +83,7 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode) ...@@ -83,7 +83,7 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode)
length = safe_snprintf(tmp, sizeof(tmp), "%2u: %s\r\n", ++answers, (char*)msg->hfield_dat[i]); length = safe_snprintf(tmp, sizeof(tmp), "%2u: %s\r\n", ++answers, (char*)msg->hfield_dat[i]);
if((p=(char*)realloc(buf,l+length+1))==NULL) { if((p=(char*)realloc(buf,l+length+1))==NULL) {
safe_snprintf(smb->last_error, sizeof(smb->last_error) safe_snprintf(smb->last_error, sizeof(smb->last_error)
,"%s realloc failure of %ld bytes for comment buffer" ,"%s realloc failure of %d bytes for comment buffer"
, __FUNCTION__, l+length+1); , __FUNCTION__, l+length+1);
free(buf); free(buf);
return(NULL); return(NULL);
...@@ -131,7 +131,7 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode) ...@@ -131,7 +131,7 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode)
continue; continue;
if((lzhbuf=(char*)malloc(length))==NULL) { if((lzhbuf=(char*)malloc(length))==NULL) {
safe_snprintf(smb->last_error, sizeof(smb->last_error) safe_snprintf(smb->last_error, sizeof(smb->last_error)
,"%s malloc failure of %ld bytes for LZH buffer" ,"%s malloc failure of %d bytes for LZH buffer"
, __FUNCTION__, length); , __FUNCTION__, length);
free(buf); free(buf);
free(preamble); free(preamble);
...@@ -139,7 +139,7 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode) ...@@ -139,7 +139,7 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode)
} }
if(smb_fread(smb,lzhbuf,length,smb->sdt_fp) != length) { if(smb_fread(smb,lzhbuf,length,smb->sdt_fp) != length) {
safe_snprintf(smb->last_error, sizeof(smb->last_error) safe_snprintf(smb->last_error, sizeof(smb->last_error)
,"%s read failure of %ld bytes for LZH data" ,"%s read failure of %d bytes for LZH data"
, __FUNCTION__, length); , __FUNCTION__, length);
free(lzhbuf); free(lzhbuf);
free(buf); free(buf);
...@@ -149,8 +149,8 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode) ...@@ -149,8 +149,8 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode)
lzhlen=*(int32_t*)lzhbuf; lzhlen=*(int32_t*)lzhbuf;
if((p=(char*)realloc(buf,l+lzhlen+3L))==NULL) { if((p=(char*)realloc(buf,l+lzhlen+3L))==NULL) {
safe_snprintf(smb->last_error, sizeof(smb->last_error) safe_snprintf(smb->last_error, sizeof(smb->last_error)
,"%s realloc failure of %ld bytes for text buffer" ,"%s realloc failure of %d bytes for text buffer"
, __FUNCTION__, l+lzhlen+3L); , __FUNCTION__, l+lzhlen+3);
free(lzhbuf); free(lzhbuf);
free(buf); free(buf);
free(preamble); free(preamble);
...@@ -164,8 +164,8 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode) ...@@ -164,8 +164,8 @@ char* smb_getmsgtxt(smb_t* smb, smbmsg_t* msg, uint mode)
else { else {
if((p=(char*)realloc(buf,l+length+3L))==NULL) { if((p=(char*)realloc(buf,l+length+3L))==NULL) {
safe_snprintf(smb->last_error, sizeof(smb->last_error) safe_snprintf(smb->last_error, sizeof(smb->last_error)
,"%s realloc failure of %ld bytes for text buffer" ,"%s realloc failure of %d bytes for text buffer"
, __FUNCTION__, l+length+3L); , __FUNCTION__, l+length+3);
free(buf); free(buf);
free(preamble); free(preamble);
return(NULL); return(NULL);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment