Changeset 1127

Show
Ignore:
Timestamp:
02/03/08 17:26:28 (1 year ago)
Author:
ahu
Message:

add some sanity checking on domain name lengths

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/pdns/pdns/packetcache.cc

    r680 r1127  
    5757 
    5858  char ckey[512]; 
    59   int len=q->qdomain.length(); 
     59  unsigned int len=q->qdomain.length(); 
     60  if(len > sizeof(ckey)) 
     61    return; 
    6062  memcpy(ckey,q->qdomain.c_str(),len); // add TOLOWER HERE FIXME XXX 
    6163  ckey[len]='|'; 
    …… 
    8890 
    8991  char ckey[512]; 
    90   int len=p.qdomain.length(); 
    91   memcpy(ckey,p.qdomain.c_str(),len); // add TOLOWER HERE FIXME XXX 
     92  unsigned int len=p.qdomain.length(); 
     93  if(len > sizeof(ckey)) 
     94    return; 
     95  memcpy(ckey, p.qdomain.c_str(), len); // add TOLOWER HERE FIXME XXX 
    9296  ckey[len]='|'; 
    9397  ckey[len+1]=packetMeritsRecursion ? 'r' : 'n';