On 12/10/2010, at 15:11, Luigi Semenzato wrote: > Thanks Rajiv. I will send you some recent logs. I have updated to > trousers 0.3.6 and testsuite 0.3. I think I have found one small > error in the TPM itself, which I will point out. >
You're welcome Luigi. Cool, let us know. The Tspi_TPM_DefineSpace04 issue is fixed in our tree, but hasn't been released in a .tar.gz yet, so isn't included in 0.3.6. > On Tue, Oct 12, 2010 at 8:26 AM, Rajiv Andrade > <[email protected]> wrote: >> Hi Luigi, sorry for the delay: >> >> On 06/10/2010, at 15:05, Luigi Semenzato wrote: >>> Tspi_NV_DefineSpace11.c: this is supposed to fail with a BADINDEX >>> error, because the D bit is set. However, trousers first consults the >>> TPM, and if the space exists it returns a TSS_E_NV_AREA_EXIST error. >>> Why is trousers doing this (and getting it wrong) instead of just >>> passing the commands through? >>> >> >> The TPM uses the same ordinal to define or release a NV area, it defines it >> if the index isn't being used, and release it in case the index is indeed >> being used. TrouSerS then retrieves the list of used indexes before >> attempting to send the TPM this very ordinal, to make sure the user isn't >> attempting to run such command on a defined index and then release it by >> accident. However, yes, TrouSerS should check if the D bit is set, and then >> return the proper error in this case, will fix this. > > Not sure I fully understand. I agree that running DefineSpace on an > existing space will delete the existing space and create a new one > (unless the size is 0) but I don't see why that should be trousers' > concern. > This is an attempt to make the usage of the TPM safer and simpler, it's very easy for an user/developer to lose track of the used indexes and release one by mistake. TrouSerS as an API is supposed to contain all the expected TPM related tasks that are application independent, this one included. Thanks, Rajiv Andrade Security Development IBM Linux Technology Center ------------------------------------------------------------------------------ Beautiful is writing same markup. Internet Explorer 9 supports standards for HTML5, CSS3, SVG 1.1, ECMAScript5, and DOM L2 & L3. Spend less time writing and rewriting code and more time creating great experiences on the web. Be a part of the beta today. http://p.sf.net/sfu/beautyoftheweb _______________________________________________ TrouSerS-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/trousers-users
