On 1/28/22 19:32, Michael Olbrich wrote:
> On Fri, Jan 28, 2022 at 06:22:19PM +0100, Christian Melki wrote:
>> On 1/28/22 17:16, Michael Olbrich wrote:
>>> On Thu, Jan 27, 2022 at 03:35:30PM +0100, Christian Melki wrote:
>>>> Approx. 2,5 year worth of fixes.
>>>> https://github.com/tpm2-software/tpm2-tss/releases
>>>> Fixes CVE-2020-24455 (three times over).
>>>> tpm2-tss now depends on JSON_C and LIBCURL.
>>>> Remove changed configuration options.
>>>
>>> Fails to build here:
>>>
>>> checking for groupadd... no
>>> configure: error: Missing required program 'groupadd': ensure it is 
>>> installed and on PATH.
>>>
>>> Adding a group should not be done during install anyways and it would fail,
>>> so I think this should work:
>>>
>>> TPM2_TSS_CONF_ENV := \
>>>     $(CROSS_ENV) \
>>>     GROUPADD=: \
>>>     USERADD=:
>>>
>>> Michael
>>>
>>
>> Didn't see that one coming. How abusive!
>> From a makefile none the less.
>> I have the required binaries on my development machine so I didn't notice.
>> Now, I've tried your suggestion, but I can't see that it helps?
>>
>> Looks like the entire thing goes about the binaries in a very static
>> fashion.
>>
>> ..
>> define make_tss_user_and_group
>>     (id -g tss 2>/dev/null || groupadd -r tss) && \
>>     (id -u tss 2>/dev/null || useradd -r -g tss tss)
>> endef
> 
>       @$(call make_tss_user_and_group) || echo "WARNING Failed to create the 
> tss user and group"
> 
> so any error here is ignored. We just need to get through the configure
> check.
> 

ack.

TPM2_TSS_CONF_ENV := \
        $(CROSS_ENV) \
        ac_cv_prog_result_groupadd=yes \
        ac_cv_prog_result_useradd=yes

should work for getting through configure?

>> ..and..
>>
>> define set_tss_permissions
>>     (chown -R tss:tss "$1") && \
>>     (chmod -R 2775 "$1") && \
>>     (setfacl -m default:group:tss:rwx "$1")
>> endef
>> ..
> 
> And 'make install' is called with fakeroot, so this will just be ignored as
> well.
> 
> Michael
> 
>>
>>>> Signed-off-by: Christian Melki <[email protected]>
>>>> ---
>>>>  rules/tpm2-tss.in   | 2 ++
>>>>  rules/tpm2-tss.make | 7 ++-----
>>>>  2 files changed, 4 insertions(+), 5 deletions(-)
>>>>
>>>> diff --git a/rules/tpm2-tss.in b/rules/tpm2-tss.in
>>>> index dbb310072..75d32188c 100644
>>>> --- a/rules/tpm2-tss.in
>>>> +++ b/rules/tpm2-tss.in
>>>> @@ -4,6 +4,8 @@ config TPM2_TSS
>>>>    tristate
>>>>    select HOST_AUTOTOOLS_AUTOCONF_ARCHIVE
>>>>    select OPENSSL
>>>> +  select JSON_C
>>>> +  select LIBCURL
>>>>    prompt "tpm2-tss"
>>>>    help
>>>>      TPM2 Software stack library - TSS and TCTI libraries
>>>> diff --git a/rules/tpm2-tss.make b/rules/tpm2-tss.make
>>>> index 768c64426..72aff609a 100644
>>>> --- a/rules/tpm2-tss.make
>>>> +++ b/rules/tpm2-tss.make
>>>> @@ -14,8 +14,8 @@ PACKAGES-$(PTXCONF_TPM2_TSS) += tpm2-tss
>>>>  #
>>>>  # Paths and names
>>>>  #
>>>> -TPM2_TSS_VERSION  := 2.3.1
>>>> -TPM2_TSS_MD5              := 5744b10e3cef56df5d65b50f51fb3fe9
>>>> +TPM2_TSS_VERSION  := 3.1.0
>>>> +TPM2_TSS_MD5              := 4d04cf52fff4ee061bb3f7b4f4ea03b7
>>>>  TPM2_TSS          := tpm2-tss-$(TPM2_TSS_VERSION)
>>>>  TPM2_TSS_SUFFIX           := tar.gz
>>>>  TPM2_TSS_URL              := 
>>>> https://github.com/tpm2-software/tpm2-tss/releases/download/$(TPM2_TSS_VERSION)/$(TPM2_TSS).$(TPM2_TSS_SUFFIX)
>>>> @@ -36,9 +36,6 @@ TPM2_TSS_CONF_OPT        := \
>>>>    $(CROSS_AUTOCONF_USR) \
>>>>    --enable-debug=info \
>>>>    --disable-unit \
>>>> -  --enable-esapi \
>>>> -  --disable-tcti-device-async \
>>>> -  --disable-tcti-partial-reads \
>>>>    --enable-tcti-device \
>>>>    --enable-tcti-mssim \
>>>>    --disable-tcti-fuzzing \
>>>> -- 
>>>> 2.30.2
>>>>
>>>>
>>>> _______________________________________________
>>>> ptxdist mailing list
>>>> [email protected]
>>>> To unsubscribe, send a mail with subject "unsubscribe" to 
>>>> [email protected]
>>>>
>>>
>>
>>
> 


_______________________________________________
ptxdist mailing list
[email protected]
To unsubscribe, send a mail with subject "unsubscribe" to 
[email protected]

Reply via email to