added check for malloc success in packet-parse.c

git-svn-id: http://svn.code.sf.net/p/retroshare/code/trunk@6950 b45a01b8-16f6-495d-af2f-9b41ad6348cc
This commit is contained in:
csoler 2013-12-19 22:16:17 +00:00
parent 2f188b8217
commit a4a4d3bdda

View File

@ -1302,6 +1302,12 @@ static int parse_user_id(ops_region_t *region,ops_parse_info_t *pinfo)
C.user_id.user_id=malloc(region->length+1); /* XXX should we not like check malloc's return value? */ C.user_id.user_id=malloc(region->length+1); /* XXX should we not like check malloc's return value? */
if(C.user_id.user_id==NULL)
{
fprintf(stderr,"malloc failed in parse_user_id") ;
return 0 ;
}
if(region->length && !limited_read(C.user_id.user_id,region->length,region, if(region->length && !limited_read(C.user_id.user_id,region->length,region,
pinfo)) pinfo))
return 0; return 0;
@ -1518,6 +1524,12 @@ static int parse_one_signature_subpacket(ops_signature_t *sig,
C.ss_raw.tag=content.tag; C.ss_raw.tag=content.tag;
C.ss_raw.length=subregion.length-1; C.ss_raw.length=subregion.length-1;
C.ss_raw.raw=malloc(C.ss_raw.length); C.ss_raw.raw=malloc(C.ss_raw.length);
if(C.ss_raw.raw==NULL)
{
fprintf(stderr,"malloc failed in parse_signature_subpackets") ;
return 0 ;
}
if(!limited_read(C.ss_raw.raw,C.ss_raw.length,&subregion,pinfo)) if(!limited_read(C.ss_raw.raw,C.ss_raw.length,&subregion,pinfo))
return 0; return 0;
CBP(pinfo,OPS_PTAG_RAW_SS,&content); CBP(pinfo,OPS_PTAG_RAW_SS,&content);