trousers Log


Commit Date  
[a6df7c] (TROUSERS_0_3_13) by Richard Maciel Richard Maciel

Bumped version to 0.3.13

2014-04-24 17:57:28 Tree
[a575f3] by Richard Maciel Richard Maciel

Added simple code to close server socket(s)

Server sockets are now properly closed before the daemon stops
executing.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-04-24 17:31:16 Tree
[4b9f79] by Richard Maciel Richard Maciel

Removed dead code from tpm_rsp_parse function

Related to coverity CID 10288.

switch case TPM_ORD_ActivateIdentity included an if structure to check
for auth1 and auth2 values. However, auth2 was checked in a previous
if. That made the if (auth2) check the default case in the structure
and the else would never be reached.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-04-09 14:53:23 Tree
[f3159b] by Richard Maciel Richard Maciel

Prevents dereference of null when call TCSP_NV_ReadValueAuth_Internal

Related coverity CID 10289

tcs_wrap_NV_ReadValueAuth can call TCSP_NV_ReadValueAuth_Internal
with a null auth. However, the latter was dereferencing the pointer
var containing the auth data without checking it, which possibly could
cause a dereference null error.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-04-08 20:04:53 Tree
[2d48f1] by Richard Maciel Richard Maciel

Prevents dereference of null when call TCSP_NV_WriteValueAuth_Internal

Related to coverity CID: 10290

pAuth variable could set to NULL in some cases, which, when passed
to the aforementioned function would result in a dereference of NULL.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-04-08 15:06:28 Tree
[fc50f5] by Richard Maciel Richard Maciel

Prevents dereference of null when calling TCTP_SetCapability_Internal

Related to Coverity CID 10291

The aforementioned function dereferenced the auth parameter without
checking for NULL first.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-04-07 16:32:37 Tree
[11c3f5] by Richard Maciel Richard Maciel

Fixed memory leak in function Transport_TerminateHandle

Related to coverity CID 10307.

Pointer handles got memory allocated for it, but that memory is never
freed at the end of the function.
Note that since obj_context_transport_execute can also allocate memory
(through the same parameter handles), it's necessary to keep track of
both the address which goes in and the address which cames out.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-03-20 19:52:06 Tree
[3e68d0] by Richard Maciel Richard Maciel

Release delfamily_list lock in the case the object found is the tpm one.

Related to coverity CID 10298.

If function obj_delfamily_find_by_familyid code execution path is
the one where an object found is of the TPM type, then the
delfamily_list lock is never released.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-03-20 16:42:58 Tree
[6c9dae] by Richard Maciel Richard Maciel

Removing possibility of NULL-dereferencing pointer

Related to coverity CID 10304.

There was a possible code execution path, in function context_destroy
that have toKill pointer var with the NULL value.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-03-20 14:40:18 Tree
[87cd1a] by Richard Maciel Richard Maciel

Removed dangled global var sd and fixed test that wrongfully used it

Related to coverity CID 10302

A global variable sd (which shouldn't exist, thus was removed) was
used as a consequence of a programming typo. The right variable to
be used is sd6.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-03-17 19:54:48 Tree
[5cce9e] by Richard Maciel Richard Maciel

Fixed memory leak in tcs_wrap_KeyControlOwner

Related to coverity CID 10310.

rgbPublicKey member wasn't being released after the return of the
function (success or failure paths).

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-03-17 19:52:17 Tree
[41201d] by Richard Maciel Richard Maciel

Fixed memory leak in ima_get_entry and fixed memset boundaries

Related to coverity CID 10311.

In some error cases the memory allocated wasn't being properly released,
so I grouped all the release in the end of the function and make error
cases point to the label there.

Also fixed a memset call which used a size 1-byte less than the
correct amount.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-03-17 18:01:47 Tree
[235630] by Richard Maciel Richard Maciel

Added check to prevent buffer overflow in name buffer.

Since the size of the name could be read from a file, but the buffer
to contain it was fixed size, a check was needed to ensure that
the fread doesn't overrun the buffer.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-03-14 12:42:50 Tree
[128ed1] by Richard Maciel Richard Maciel

Fixed code reading data from freed pointer.

Related to coverit CID 10331.

free call being executed over pointer, before the
LogError which used it.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-04-09 18:18:00 Tree
[0d926c] by Richard Maciel Richard Maciel

Fixed modules where unitialized pointers could be freed

Related to coverit CIDs 10326 and 10323

In both cases proper pointer initialization was not
made, so, in some cases, the code could free the value of a
unitialized pointer.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-04-09 18:16:45 Tree
[c386ce] by Richard Maciel Richard Maciel

Inserted missing va_end

Related to coverit CIDs 10332 and 10333.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-04-09 16:31:13 Tree
[be1e5a] by Richard Maciel Richard Maciel

memcmp result was being trunc'ed before the test

Related to coverit CID 10255.

Return value of memcmp was being converted to TSS_BOOL type before
being tested, thus possibly missing information (since it can return
an integer value != of zero, depending on the differences found).

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-03-11 13:17:29 Tree
[6462e4] by Richard Maciel Richard Maciel

Unload_KM_KEYINFO called UnloadBlob_BYTE passing parameters in wrong order

Related to coverit CID 10254.

Blob parameter is the last one instead of the middle one.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-03-11 12:06:35 Tree
[1ebb61] by Richard Maciel Richard Maciel

Finally added Richard Maciel as the current maintainer.

Richard started as TrouSerS maintainer since August, 2013,
and is currently the single maintainer of this project.

2014-03-10 15:19:43 Tree
[88d6fb] by Richard Maciel Richard Maciel

Don't allow TrouSerS to wrap asymmetric key > 2048 bits

Previously the tspi layer accepted external asymmetric keys of any
size to be wrapped by the TPM, however it allocated buffers capable
of only holding private keys with at most 2048 bits.

A test was added to prevent this behavior, since the TPM 1.1b and
1.2 limits that size, according to the TPM_STORE_ASYMKEY structure.

Also, comments and defines were added to make the life of future
maintainers easier when debugging that function.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2014-02-25 13:55:23 Tree
[7d2072] by Richard Maciel Richard Maciel

Remove references to my_init

Patch submitted by Nikos Mavrogiannopoulos (using sourceforge bugzilla)
to make the my_init function static because it was causing conflicts with
MySQL.

2014-03-03 16:38:29 Tree
[48ed84] by Richard Maciel Richard Maciel

Removed references to 'list_init'

All macros that previously referred to list_init, now refers to
tspi_list_init.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>
Acked-by:Joel Schopp <jschopp@linux.vnet.ibm.com>

2014-02-20 16:57:04 Tree
[43f282] by Richard Maciel Richard Maciel

list_init symbol got a different name and is now local-only

list_init function in src/tspi/obj.c had a name too generic
and it was being exported which was causing issues with other
libraries.

Applying both solutions is unnecessary, but I think it needs to
have a more specific name.

Signed-off-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>
Acked-by:Joel Schopp <jschopp@linux.vnet.ibm.com>

2014-02-20 16:57:03 Tree
[c6fe2d] by Ceri Coburn Ceri Coburn , pushed by Richard Maciel Richard Maciel

Fixed 2 memory leaks inside Tspi_Key_CertifyKey, CertifyInfo and outData structures

Signed-off-by: Ceri Coburn <ceri.coburn@gmail.com>
Acked-by: Joel Schopp <jschopp@linux.vnet.ibm.com>
Tested-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2013-10-24 09:42:41 Tree
[971468] by Ceri Coburn Ceri Coburn , pushed by Richard Maciel Richard Maciel

Memory leak fix for Tspi_TPM_CollateIdentityRequest

Signed-off-by: Ceri Coburn <ceri.coburn@gmail.com>
Acked-by: Joel Schopp <jschopp@linux.vnet.ibm.com>
Tested-by: Richard Maciel <rmaciel@linux.vnet.ibm.com>

2013-10-24 09:14:45 Tree
Older >

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks