Re: newbie: set cert time validity

2007-07-17 Thread imin noob

No wonder I couldn't find the MakeCertificate function, it's actually
resides in an external library. I 'll try to do anything I could to make
sure it works as needed..

However, thank you very much for your help David Schwartz.. if you were in
java.sun.com forum I'd surely have given you at least 7 duke dollars.

Thanks again

On 7/17/07, David Schwartz <[EMAIL PROTECTED]> wrote:






thanks a lot for your lenghty explanation, David Schwartz. I really
appreciate it for you to help me explain all this. I noted you said that what
I did might be sensible if three things are the case: 1) The locale you
are using the certificate has no daylight savings time.
2) The certificate isn't going anywhere, it's only going to be used in one
place.
3) The certificate expires in the near future, so a risk of a change in
daylight savings time rules is low.

For the no (1), i'm not really sure about this daylight savings time.. I
reside in Malaysia (next to singapore and thailand) and I'm not sure whether
my country has any daylight savings time or not. For no(2), currently the
issued certificates is only used in our office.

I don't know the daylight savings time rules in your area, but if you have
daylight savings time, then some of your certificates will expire an hour
off from when you intended. I'm pretty sure you do have a daylight savings
time and certificates issued that expire during daylight savings time will
not actually expire at midnight but will be one hour off.

if ((x = MakeCertificate(req,sconf,nconf,NULL,ca,ca_pkey,
pinfo.begin_validity,
 pinfo.validity,pinfo.serial,pinfo.algo,0)) ==
NULL)
{  ret = ERROR_MAKECERT;
   goto end3;
}

X509_gmtime_adj(X509_get_notBefore(x),0); //added on 16/7/2007
X509_gmtime_adj(X509_get_notAfter(x),(long)60*60*24*pinfo.validity);
//added on 16/7/2007
X509_gmtime_roundup(X509_get_notAfter(x)); //added on 16/7/2007

This is wrong, you cannot modify the certificate after it is signed. You
have to modify the 'MakeCertificate' function.

As you may see above, I added the X509_gmtime_adj and X509_gmtime_roundup
after the call to makeCertificate. The generated certificate will have the
desired expiry date, but the cert itself would be corrupted. It will have
this message displayed in the cert - "The integrity of this certificate
cannot be guaranteed. THe certificate may be corrupted or may have been
altered." I guess this happens because I added the line X509_gmtime_ after
the cert has been created, right? But I don't know anyway else where I
should put it.

Inside the 'MakeCertificate' function.

And for the MakeCertificate function which was called above, all I could
find was this code:

 X509 *MakeCertificate(X509_REQ *preq,char **sconf,int nconf,EVP_PKEY
*self_key,
  char* cacert_file,EVP_PKEY *ca_key,int
pbegin,int pdays,
  long pserial,int palgo,int ca_type)

which was located in Global.h file. It doesn't seem any where I can put
the X509_gmtime_roundup line

Where is the code to the 'MakeCertificate' function? That where you'll
have to do it.

DS





Re: newbie: set cert time validity

2007-07-16 Thread imin noob

thanks a lot for your lenghty explanation, David Schwartz. I really
appreciate it for you to help me explain all this. I noted you said that what
I did might be sensible if three things are the case: 1) The locale you are
using the certificate has no daylight savings time.
2) The certificate isn't going anywhere, it's only going to be used in one
place.
3) The certificate expires in the near future, so a risk of a change in
daylight savings time rules is low.

For the no (1), i'm not really sure about this daylight savings time.. I
reside in Malaysia (next to singapore and thailand) and I'm not sure whether
my country has any daylight savings time or not. For no(2), currently the
issued certificates is only used in our office.

Anyway to issue certificate, the codes is as below:

if
(!(returnIssueCertificate=IssueCertificate(cinfo,x509,skey,ca,Type,&HexSerial,sendUserName,sendUserID)))
   {
   MessageDlg("Issue certificate is
failed",mtError,TMsgDlgButtons()serialNumber,buf1);
   *HexSerial = (AnsiString)buf1;



   CVAULT_X509_write(x,x509);
   MakePKCS12(pass.c_str(),name.c_str(),pkey,x,p12Path.c_str());
   s = progpath + "client.key";
   WriteKey(pkey,s.c_str());

   //s = progpath + "user.crt";
   s = progpath + receiveUserID + "-" + receiveUserName + ".crt";
   if ((fp = fopen(s.c_str(),"w")) == NULL)
   {  ShowMessage("ERROR: Open cert.crt");
   return -3; //to indicate that program unable to open user.crt
   }
   fprintf(fp,"%s",x509);
   fclose(fp);


   // Write CA certificate
   if ((xca = ReadCertificate(ca)) == NULL)
   {  ret = ERROR_READ_CACERT;
  goto end3;
   }

   CVAULT_X509_write(xca,cacert);
   s = progpath + "CA.crt";
   if ((fp = fopen(s.c_str(),"w")) == NULL)
   {  ShowMessage("ERROR: Open CA.crt");
  return -1;
   }
   fprintf(fp,"%s",cacert);
   fclose(fp);
   X509_free(xca);

   ret = 1;
   //MainForm->tinfo = pinfo;

end3:
   X509_REQ_free(req);

end2:
   EVP_PKEY_free(pkey);

end1:

   for (i=0; i<80; i++)
  free(sconf[i]);
   //endfor

   return ret;
}

As you may see above, I added the X509_gmtime_adj and X509_gmtime_roundup
after the call to makeCertificate. The generated certificate will have the
desired expiry date, but the cert itself would be corrupted. It will have
this message displayed in the cert - "The integrity of this certificate
cannot be guaranteed. THe certificate may be corrupted or may have been
altered." I guess this happens because I added the line X509_gmtime_ after
the cert has been created, right? But I don't know anyway else where I
should put it.


And for the MakeCertificate function which was called above, all I could
find was this code:

X509 *MakeCertificate(X509_REQ *preq,char **sconf,int nconf,EVP_PKEY
*self_key,
 char* cacert_file,EVP_PKEY *ca_key,int pbegin,int
pdays,
 long pserial,int palgo,int ca_type)

which was located in Global.h file. It doesn't seem any where I can put the
X509_gmtime_roundup line

*Please don't take this the wrong way -- but you are modifying
security-critical code based on a requirement that seems to make no sense.
I've told the management of my company that I don't want to continue
debugging this code, but they insist I have to do it because they have no
one else to do it... yes, lame reason from them, but I'm in no position to
say no. anyway I guess if this software is broken, they're the one who
should be blamed.. bcos i've told them I don't want to continue doing
this...


On 7/16/07, David Schwartz <[EMAIL PROTECTED]> wrote:





hold on! thanks a lot I managed to get it to 23:59:59. all i had to do was
change the value
strcpy(buf+6, "235959Z"); to  strcpy(buf+6, "155959Z");

I would not do that. There is no way you can know that 15:59:59 will
correspond to 24:59:59 in the future when the certificate expires. You
are essentially predicting what the time zone shift will be at some future
date. I would strongly urge you to make it expire at midnight UTC/GMT time.

I would go further as to say that whatever tool is presenting certificate
expiration times to you as '1/8/2007 7:59:59' (which is the way you pasted
it) should be dumped and replaced with something sane. This contains no time
zone indicator or GMT offset. If you paste it to a mailing list, it is
meaningless.

If your requirement really is that a certificate expire at midnight for
the time zone in which it was issued, assuming the zone offset will be the
same at certificate issue time as it was at certificate issue time, then
the requirement should be re-examined.  For one thing, '155959Z' can't
possibly be right for every possible case (unless your locality has no
daylight savings time and you get lucky and it never does).
You are assuming that 15:59:59 local time will correspond to 24:59:59 UTC
time at the time and place the certificate is being used when it expires.
This seems like a truly crazy assumption. It might be sensible if three
things are the case:
1) The locale