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

Address Coverity-scan reported issues 

Could use more long->off_t conversions, but this is a start.
parent eb160a3f
No related branches found
No related tags found
1 merge request!463MRC mods by Codefenix (2024-10-20)
Pipeline #2822 passed
Stage: build
Show whitespace changes
Inline Side-by-side
Showing
with 49 additions and 43 deletions
src/sbbs3/userdat.c
+ 49
− 43
View file @ d4306822
...@@ -54,7 +54,7 @@ uint matchuser(scfg_t* cfg, const char *name, BOOL sysop_alias) ...@@ -54,7 +54,7 @@ uint matchuser(scfg_t* cfg, const char *name, BOOL sysop_alias)
int file,c; int file,c;
char dat[LEN_ALIAS+2]; char dat[LEN_ALIAS+2];
char str[256]; char str[256];
ulong l,length; off_t l,length;
FILE* stream; FILE* stream;
if(!VALID_CFG(cfg) || name==NULL) if(!VALID_CFG(cfg) || name==NULL)
...@@ -67,9 +67,13 @@ uint matchuser(scfg_t* cfg, const char *name, BOOL sysop_alias) ...@@ -67,9 +67,13 @@ uint matchuser(scfg_t* cfg, const char *name, BOOL sysop_alias)
SAFEPRINTF(str,"%suser/name.dat",cfg->data_dir); SAFEPRINTF(str,"%suser/name.dat",cfg->data_dir);
if((stream=fnopen(&file,str,O_RDONLY))==NULL) if((stream=fnopen(&file,str,O_RDONLY))==NULL)
return(0); return(0);
length=(long)filelength(file); length = filelength(file);
for(l=0;l<length;l+=LEN_ALIAS+2) { if(length < sizeof(dat)) {
fread(dat,sizeof(dat),1,stream); fclose(stream);
return 0;
}
for(l = 0; l < length; l += sizeof(dat)) {
(void)fread(dat,sizeof(dat),1,stream);
for(c=0;c<LEN_ALIAS;c++) for(c=0;c<LEN_ALIAS;c++)
if(dat[c]==ETX) break; if(dat[c]==ETX) break;
dat[c]=0; dat[c]=0;
...@@ -78,7 +82,7 @@ uint matchuser(scfg_t* cfg, const char *name, BOOL sysop_alias) ...@@ -78,7 +82,7 @@ uint matchuser(scfg_t* cfg, const char *name, BOOL sysop_alias)
} }
fclose(stream); fclose(stream);
if(l<length) if(l<length)
return((l/(LEN_ALIAS+2))+1); return (uint)((l/(LEN_ALIAS+2))+1);
return(0); return(0);
} }
...@@ -127,8 +131,9 @@ uint total_users(scfg_t* cfg) ...@@ -127,8 +131,9 @@ uint total_users(scfg_t* cfg)
if((file=openuserdat(cfg, /* for_modify: */FALSE)) < 0) if((file=openuserdat(cfg, /* for_modify: */FALSE)) < 0)
return(0); return(0);
length=(long)filelength(file); length=(long)filelength(file);
if(length >= U_LEN) {
for(l=0;l<length;l+=U_LEN) { for(l=0;l<length;l+=U_LEN) {
lseek(file,l+U_MISC,SEEK_SET); (void)lseek(file,l+U_MISC,SEEK_SET);
if(read(file,str,8)!=8) if(read(file,str,8)!=8)
continue; continue;
getrec(str,0,8,str); getrec(str,0,8,str);
...@@ -136,6 +141,7 @@ uint total_users(scfg_t* cfg) ...@@ -136,6 +141,7 @@ uint total_users(scfg_t* cfg)
continue; continue;
total_users++; total_users++;
} }
}
close(file); close(file);
return(total_users); return(total_users);
} }
...@@ -227,7 +233,7 @@ int readuserdat(scfg_t* cfg, unsigned user_number, char* userdat, int infile) ...@@ -227,7 +233,7 @@ int readuserdat(scfg_t* cfg, unsigned user_number, char* userdat, int infile)
return(-1); /* no such user record */ return(-1); /* no such user record */
} }
lseek(file,(long)((long)(user_number-1)*U_LEN),SEEK_SET); (void)lseek(file,(long)((long)(user_number-1)*U_LEN),SEEK_SET);
i=0; i=0;
while(i<LOOP_NODEDAB while(i<LOOP_NODEDAB
&& lock(file,(long)((long)(user_number-1)*U_LEN),U_LEN)==-1) { && lock(file,(long)((long)(user_number-1)*U_LEN),U_LEN)==-1) {
...@@ -615,7 +621,7 @@ int putuserdat(scfg_t* cfg, user_t* user) ...@@ -615,7 +621,7 @@ int putuserdat(scfg_t* cfg, user_t* user)
return(-4); return(-4);
} }
lseek(file,(long)((long)((long)user->number-1)*U_LEN),SEEK_SET); (void)lseek(file,(long)((long)((long)user->number-1)*U_LEN),SEEK_SET);
i=0; i=0;
while(i<LOOP_NODEDAB while(i<LOOP_NODEDAB
...@@ -673,8 +679,8 @@ char* username(scfg_t* cfg, int usernumber, char *name) ...@@ -673,8 +679,8 @@ char* username(scfg_t* cfg, int usernumber, char *name)
name[0]=0; name[0]=0;
return(name); return(name);
} }
lseek(file,(long)((long)(usernumber-1)*(LEN_ALIAS+2)),SEEK_SET); (void)lseek(file,(long)((long)(usernumber-1)*(LEN_ALIAS+2)),SEEK_SET);
read(file,name,LEN_ALIAS); (void)read(file,name,LEN_ALIAS);
close(file); close(file);
for(c=0;c<LEN_ALIAS;c++) for(c=0;c<LEN_ALIAS;c++)
if(name[c]==ETX) break; if(name[c]==ETX) break;
...@@ -692,8 +698,8 @@ int putusername(scfg_t* cfg, int number, const char *name) ...@@ -692,8 +698,8 @@ int putusername(scfg_t* cfg, int number, const char *name)
char str[256]; char str[256];
int file; int file;
int wr; int wr;
long length; off_t length;
uint total_users; off_t total_users;
if(!VALID_CFG(cfg) || name==NULL || number<1) if(!VALID_CFG(cfg) || name==NULL || number<1)
return(-1); return(-1);
...@@ -701,12 +707,12 @@ int putusername(scfg_t* cfg, int number, const char *name) ...@@ -701,12 +707,12 @@ int putusername(scfg_t* cfg, int number, const char *name)
SAFEPRINTF(str,"%suser/name.dat", cfg->data_dir); SAFEPRINTF(str,"%suser/name.dat", cfg->data_dir);
if((file=nopen(str,O_RDWR|O_CREAT))==-1) if((file=nopen(str,O_RDWR|O_CREAT))==-1)
return(errno); return(errno);
length=(long)filelength(file); length = filelength(file);
/* Truncate corrupted name.dat */ /* Truncate corrupted name.dat */
total_users=lastuser(cfg); total_users=lastuser(cfg);
if((uint)(length/(LEN_ALIAS+2))>total_users) if(length/(LEN_ALIAS+2) > total_users)
chsize(file,total_users*(LEN_ALIAS+2)); chsize(file,(long)(total_users*(LEN_ALIAS+2)));
if(length && length%(LEN_ALIAS+2)) { if(length && length%(LEN_ALIAS+2)) {
close(file); close(file);
...@@ -715,11 +721,11 @@ int putusername(scfg_t* cfg, int number, const char *name) ...@@ -715,11 +721,11 @@ int putusername(scfg_t* cfg, int number, const char *name)
if(length<(((long)number-1)*(LEN_ALIAS+2))) { if(length<(((long)number-1)*(LEN_ALIAS+2))) {
SAFEPRINTF2(str,"%*s\r\n",LEN_ALIAS,nulstr); SAFEPRINTF2(str,"%*s\r\n",LEN_ALIAS,nulstr);
memset(str,ETX,LEN_ALIAS); memset(str,ETX,LEN_ALIAS);
lseek(file,0L,SEEK_END); (void)lseek(file,0L,SEEK_END);
while(filelength(file)<((long)number*(LEN_ALIAS+2))) while((length = filelength(file)) >= 0 && length < ((long)number*(LEN_ALIAS+2))) // Shouldn't this be (number-1)?
write(file,str,(LEN_ALIAS+2)); (void)write(file,str,(LEN_ALIAS+2));
} }
lseek(file,(long)(((long)number-1)*(LEN_ALIAS+2)),SEEK_SET); (void)lseek(file,(long)(((long)number-1)*(LEN_ALIAS+2)),SEEK_SET);
putrec(str,0,LEN_ALIAS,name); putrec(str,0,LEN_ALIAS,name);
putrec(str,LEN_ALIAS,2,crlf); putrec(str,LEN_ALIAS,2,crlf);
wr=write(file,str,LEN_ALIAS+2); wr=write(file,str,LEN_ALIAS+2);
...@@ -919,7 +925,7 @@ int getnodedat(scfg_t* cfg, uint number, node_t *node, BOOL lockit, int* fdp) ...@@ -919,7 +925,7 @@ int getnodedat(scfg_t* cfg, uint number, node_t *node, BOOL lockit, int* fdp)
for(count=0;count<LOOP_NODEDAB;count++) { for(count=0;count<LOOP_NODEDAB;count++) {
if(count) if(count)
mswait(100); mswait(100);
lseek(file,(long)number*sizeof(node_t),SEEK_SET); (void)lseek(file,(long)number*sizeof(node_t),SEEK_SET);
if(lockit if(lockit
&& lock(file,(long)number*sizeof(node_t),sizeof(node_t))!=0) && lock(file,(long)number*sizeof(node_t),sizeof(node_t))!=0)
continue; continue;
...@@ -962,7 +968,7 @@ int putnodedat(scfg_t* cfg, uint number, node_t* node, BOOL closeit, int file) ...@@ -962,7 +968,7 @@ int putnodedat(scfg_t* cfg, uint number, node_t* node, BOOL closeit, int file)
number--; /* make zero based */ number--; /* make zero based */
for(attempts=0;attempts<10;attempts++) { for(attempts=0;attempts<10;attempts++) {
lseek(file,(long)number*sizeof(node_t),SEEK_SET); (void)lseek(file,(long)number*sizeof(node_t),SEEK_SET);
if((wr=write(file,node,sizeof(node_t)))==sizeof(node_t)) if((wr=write(file,node,sizeof(node_t)))==sizeof(node_t))
break; break;
wrerr=errno; /* save write error */ wrerr=errno; /* save write error */
...@@ -1065,8 +1071,8 @@ char* getnodeext(scfg_t* cfg, int num, char* buf) ...@@ -1065,8 +1071,8 @@ char* getnodeext(scfg_t* cfg, int num, char* buf)
return ""; return "";
if((f = opennodeext(cfg)) < 1) if((f = opennodeext(cfg)) < 1)
return ""; return "";
lseek(f, (num-1) * 128, SEEK_SET); (void)lseek(f, (num-1) * 128, SEEK_SET);
read(f, buf, 128); (void)read(f, buf, 128);
close(f); close(f);
buf[127] = 0; buf[127] = 0;
return buf; return buf;
...@@ -1330,7 +1336,7 @@ uint userdatdupe(scfg_t* cfg, uint usernumber, uint offset, uint datlen ...@@ -1330,7 +1336,7 @@ uint userdatdupe(scfg_t* cfg, uint usernumber, uint offset, uint datlen
progress(cbdata, l, length); progress(cbdata, l, length);
if(usernumber && l/U_LEN==(long)usernumber-1) if(usernumber && l/U_LEN==(long)usernumber-1)
continue; continue;
lseek(file,l+offset,SEEK_SET); (void)lseek(file,l+offset,SEEK_SET);
i=0; i=0;
while(i<LOOP_NODEDAB && lock(file,l,U_LEN)==-1) { while(i<LOOP_NODEDAB && lock(file,l,U_LEN)==-1) {
if(i) if(i)
...@@ -1343,15 +1349,15 @@ uint userdatdupe(scfg_t* cfg, uint usernumber, uint offset, uint datlen ...@@ -1343,15 +1349,15 @@ uint userdatdupe(scfg_t* cfg, uint usernumber, uint offset, uint datlen
return(0); return(0);
} }
read(file,str,datlen); (void)read(file,str,datlen);
for(i=0;i<datlen;i++) for(i=0;i<datlen;i++)
if(str[i]==ETX) break; if(str[i]==ETX) break;
str[i]=0; str[i]=0;
truncsp(str); truncsp(str);
if(!stricmp(str,dat)) { if(!stricmp(str,dat)) {
if(!del) { /* Don't include deleted users in search */ if(!del) { /* Don't include deleted users in search */
lseek(file,l+U_MISC,SEEK_SET); (void)lseek(file,l+U_MISC,SEEK_SET);
read(file,str,8); (void)read(file,str,8);
getrec(str,0,8,str); getrec(str,0,8,str);
if(ahtoul(str)&(DELETED|INACTIVE)) { if(ahtoul(str)&(DELETED|INACTIVE)) {
unlock(file,l,U_LEN); unlock(file,l,U_LEN);
...@@ -1503,7 +1509,7 @@ char* getnmsg(scfg_t* cfg, int node_num) ...@@ -1503,7 +1509,7 @@ char* getnmsg(scfg_t* cfg, int node_num)
if((file=nopen(str,O_RDWR))==-1) if((file=nopen(str,O_RDWR))==-1)
return(NULL); return(NULL);
length=(long)filelength(file); length=(long)filelength(file);
if(!length) { if(length < 1) {
close(file); close(file);
return(NULL); return(NULL);
} }
...@@ -1541,7 +1547,7 @@ int putnmsg(scfg_t* cfg, int num, char *strin) ...@@ -1541,7 +1547,7 @@ int putnmsg(scfg_t* cfg, int num, char *strin)
SAFEPRINTF2(str,"%smsgs/n%3.3u.msg",cfg->data_dir,num); SAFEPRINTF2(str,"%smsgs/n%3.3u.msg",cfg->data_dir,num);
if((file=nopen(str,O_WRONLY|O_CREAT))==-1) if((file=nopen(str,O_WRONLY|O_CREAT))==-1)
return(errno); return(errno);
lseek(file,0L,SEEK_END); /* Instead of opening with O_APPEND */ (void)lseek(file,0L,SEEK_END); /* Instead of opening with O_APPEND */
i=strlen(strin); i=strlen(strin);
if(write(file,strin,i)!=i) { if(write(file,strin,i)!=i) {
close(file); close(file);
...@@ -2198,7 +2204,7 @@ int getuserrec(scfg_t* cfg, int usernumber,int start, int length, char *str) ...@@ -2198,7 +2204,7 @@ int getuserrec(scfg_t* cfg, int usernumber,int start, int length, char *str)
close(file); close(file);
return(-2); return(-2);
} }
lseek(file,(long)((long)(usernumber-1)*U_LEN)+start,SEEK_SET); (void)lseek(file,(long)((long)(usernumber-1)*U_LEN)+start,SEEK_SET);
if(length < 1) { /* auto-length */ if(length < 1) { /* auto-length */
length=user_rec_len(start); length=user_rec_len(start);
...@@ -2332,7 +2338,7 @@ ulong adjustuserrec(scfg_t* cfg, int usernumber, int start, int length, long adj ...@@ -2332,7 +2338,7 @@ ulong adjustuserrec(scfg_t* cfg, int usernumber, int start, int length, long adj
return(0); return(0);
} }
lseek(file,(long)((long)(usernumber-1)*U_LEN)+start,SEEK_SET); (void)lseek(file,(long)((long)(usernumber-1)*U_LEN)+start,SEEK_SET);
if(length < 1) { /* auto-length */ if(length < 1) { /* auto-length */
length=user_rec_len(start); length=user_rec_len(start);
...@@ -2381,7 +2387,7 @@ ulong adjustuserrec(scfg_t* cfg, int usernumber, int start, int length, long adj ...@@ -2381,7 +2387,7 @@ ulong adjustuserrec(scfg_t* cfg, int usernumber, int start, int length, long adj
sval += (ushort)adj; sval += (ushort)adj;
val = sval; val = sval;
} }
lseek(file,(long)((long)(usernumber-1)*U_LEN)+start,SEEK_SET); (void)lseek(file,(long)((long)(usernumber-1)*U_LEN)+start,SEEK_SET);
putrec(str,0,length,ultoa(val,tmp,10)); putrec(str,0,length,ultoa(val,tmp,10));
if(write(file,str,length)!=length) { if(write(file,str,length)!=length) {
unlock(file,(long)((long)(usernumber-1)*U_LEN)+start,length); unlock(file,(long)((long)(usernumber-1)*U_LEN)+start,length);
...@@ -2845,11 +2851,11 @@ int newuserdat(scfg_t* cfg, user_t* user) ...@@ -2845,11 +2851,11 @@ int newuserdat(scfg_t* cfg, user_t* user)
if((file=nopen(str,O_RDWR))==-1) if((file=nopen(str,O_RDWR))==-1)
continue; continue;
memset(&stats,0,sizeof(stats)); memset(&stats,0,sizeof(stats));
lseek(file,4L,SEEK_SET); /* Skip timestamp */ (void)lseek(file,4L,SEEK_SET); /* Skip timestamp */
read(file,&stats,sizeof(stats)); (void)read(file,&stats,sizeof(stats));
stats.nusers++; stats.nusers++;
lseek(file,4L,SEEK_SET); (void)lseek(file,4L,SEEK_SET);
write(file,&stats,sizeof(stats)); (void)write(file,&stats,sizeof(stats));
close(file); close(file);
} }
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment