Date   

[Bug 4082] CryptoPkg: Add more Crypto X509 funtions.

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4082

qi1.zhang@... <qi1.zhang@...> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |CONFIRMED
Assignee|unassigned@... |qi1.zhang@...
Ever confirmed|0 |1

--- Comment #1 from qi1.zhang@... <qi1.zhang@...> ---
WIP

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4082] New: CryptoPkg: Add more Crypto X509 funtions.

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4082

Bug ID: 4082
Summary: CryptoPkg: Add more Crypto X509 funtions.
Product: EDK2
Version: Current
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: normal
Priority: Lowest
Component: Code
Assignee: unassigned@...
Reporter: qi1.zhang@...
CC: edk2+bugs+int+994+563148131503455288@groups.io

Need to add more Crypto X509 funtions.

X509GetVersion;
X509GetSerialNumber;
X509GetIssuerName;
X509GetSignatureAlgorithm;
X509GetExtensionData;
X509GetExtendedKeyUsage;
X509GetValidity;
X509SetDateTime;
X509CompareDateTime;
X509GetKeyUsage;
X509VerifyCertChain;
X509GetCertFromCertChain;
Asn1GetTag;
X509GetExtendedBasicConstraints;

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4080] Add missing library mappings for CRYPTO_SERVICES set to ALL or MIN_DXE_MIN_SMM

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4080

Michael Kinney <michael.d.kinney@...> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|IN_PROGRESS |RESOLVED
Resolution|--- |FIXED

--- Comment #3 from Michael Kinney <michael.d.kinney@...> ---
Commit:

https://github.com/tianocore/edk2/commit/582a7c9995a8fd036a3583ba485467f5a0316cca

PR: https://github.com/tianocore/edk2/pull/3397

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4081] New: INF [Sources] section Feature Flag Expressions do not use override values from build --pcd option

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4081

Bug ID: 4081
Summary: INF [Sources] section Feature Flag Expressions do not
use override values from build --pcd option
Product: EDK2
Version: Current
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: normal
Priority: Lowest
Component: Code
Assignee: unassigned@...
Reporter: michael.d.kinney@...
CC: edk2+bugs+int+994+563148131503455288@groups.io

If a PCD value is set using the build command --pcd option and that
PCD is used in a Feature Flag Expression in an INF [Sources] section,
then the PCD value in the expression evaluation is from only the
DEC and DSC file values. The --pcd override is ignored.

This issue can be reproduced when building the CryptoPkg and
attempting to set PcdOpensslEcEnabled to TRUE. The build
will break with undefined symbols because the EC related source
files were not compiled into the openssl lib.

build -a IA32 -n 5 -t VS2019 -p CryptoPkg\CryptoPkg.dsc -D CRYPTO_SERVICES=ALL
--pcd gEfiCryptoPkgTokenSpaceGuid.PcdOpensslEcEnabled=TRUE

The bug is in the file BaseTools/Source/Python/Workspace/InfBuildData.py
in the function CheckFeatureFlagPcd(). It only uses
GlobalData.gPlatformFinalPcds and does not evaluate GlobalData.BuildOptionPcd
or some other source of the
PCDs set using the --pcd option.

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4080] Add missing library mappings for CRYPTO_SERVICES set to ALL or MIN_DXE_MIN_SMM

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4080

Michael Kinney <michael.d.kinney@...> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |IN_PROGRESS
Ever confirmed|0 |1

--- Comment #2 from Michael Kinney <michael.d.kinney@...> ---
Patch email review: https://edk2.groups.io/g/devel/message/94258

PR: https://github.com/tianocore/edk2/pull/3397

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4080] Add missing library mappings for CRYPTO_SERVICES set to ALL or MIN_DXE_MIN_SMM

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4080

Michael Kinney <michael.d.kinney@...> changed:

What |Removed |Added
----------------------------------------------------------------------------
Assignee|unassigned@... |michael.d.kinney@...

--- Comment #1 from Michael Kinney <michael.d.kinney@...> ---
Assign to owner

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4080] New: Add missing library mappings for CRYPTO_SERVICES set to ALL or MIN_DXE_MIN_SMM

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4080

Bug ID: 4080
Summary: Add missing library mappings for CRYPTO_SERVICES set
to ALL or MIN_DXE_MIN_SMM
Product: EDK2
Version: Current
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: normal
Priority: Lowest
Component: Code
Assignee: unassigned@...
Reporter: michael.d.kinney@...
CC: edk2+bugs+int+994+563148131503455288@groups.io

Update CryptoPkg.dsc file to add SyncronizationLib and MmServicesTableLib
mappings. Without these mappings, the SMM builds generate a build break
because the new parallel hash algorithm uses MP Services and spin locks.

The following build commands reproduce these failures:

build -a IA32 -a X64 -n 5 -t VS2019 -p CryptoPkg\CryptoPkg.dsc -D
CRYPTO_SERVICES=ALL

build -a IA32 -a X64 -n 5 -t VS2019 -p CryptoPkg\CryptoPkg.dsc -D
CRYPTO_SERVICES=MIN_DXE_MIN_SMM

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 3654] Openssl native instructions for 32-bit

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=3654

Christopher Zurcher <christopher.zurcher@...> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|CONFIRMED |RESOLVED
Resolution|--- |FIXED

--- Comment #3 from Christopher Zurcher <christopher.zurcher@...> ---
Completed:
https://github.com/tianocore/edk2/commit/03f708090b9da25909935e556c351a4d9445fd3f

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4079] New: EFI_KEYWORD_HANDLER_PROTOCOL.GetData() handle NamespaceId string as prefix

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4079

Bug ID: 4079
Summary: EFI_KEYWORD_HANDLER_PROTOCOL.GetData() handle
NamespaceId string as prefix
Product: EDK2
Version: Current
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: normal
Priority: Lowest
Component: Code
Assignee: unassigned@...
Reporter: aladyshev22@...
CC: edk2+bugs+int+994+563148131503455288@groups.io

EFI_KEYWORD_HANDLER_PROTOCOL.GetData() function receives 'EFI_STRING
NamespaceId' as one of it's arguments.

According to the UEFI specification the meaning of this argument is:
```
NamespaceId - A null-terminated string containing the platform configuration
language to search through in the system. If a NULL is passed in, then it is
assumed that any platform configuration language with the prefix of “x-UEFI-”
are searched.
```

From the description it looks like if the argument is provided, function should
look for the exact namespace match. But currently the match is performed
similar to the NULL case, i.e. the provided value is treated like the namespace
prefix.

For example these strings give the same result:
NAMESPACE=x-UEFI
NAMESPACE=x-UEFI-
NAMESPACE=x-UEFI-n
NAMESPACE=x-UEFI-ns

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4036] CryptoPkg: Add AeadAesGcm support

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4036

qi1.zhang@... <qi1.zhang@...> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|IN_PROGRESS |RESOLVED
Resolution|--- |FIXED

--- Comment #2 from qi1.zhang@... <qi1.zhang@...> ---
Merged https://github.com/tianocore/edk2/pull/3388

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4078] New: Enhancing the logic for AtaPassThruGetNextPort()/AtaPassThruGetNextDevice()

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4078

Bug ID: 4078
Summary: Enhancing the logic for
AtaPassThruGetNextPort()/AtaPassThruGetNextDevice()
Product: EDK2
Version: Current
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: normal
Priority: Lowest
Component: Code
Assignee: unassigned@...
Reporter: karunakar_poosapalli@...
CC: edk2+bugs+int+994+563148131503455288@groups.io

1. Based on current Design of AtaPassThruGetNextDevice , the caller of
GetNextDevice must wait until status is EFI_NOT_FOUND.
2. If any calling driver or application calls AtaPassThru->GetNextDevice() and
didn't wait until Status is EFI_NOT_FOUND and calls GetNextDevice() only for
success case(Calling only once),
the AtaPassThruGetNextDevice logic will effect next immediate calling driver
with returning EFI_NOT_FOUND as previous call was ended with success case
itself with setiting Instance->PreviousPortMultiplier = 0xFFFF

3. Based on AtaPassThruGetNextDevice current design it expects calling driver
or application must call GetNextDevice() unitl it returns EFI_NOT_FOUND
But there are some examples where If a calling driver is interested in say
WD disk among all Micron disks, driver can continue the
GetNextPort()->GetNextDevice()->GetIdentifyData() loop until it finds WD and
just break away.
Driver can not be mandated to look into all ports/devices further. Right ?

Because of test driver/Application or a module which intended to call
GetNextDevice() only once, Because of this next drivers using GetNextDevice()
should not get impacted and get invalid behaviour right?


As the driver flow or modules dispaching can’t just remember a
Port/PortMultiplierPort and randomly call GetNextPort()->GetNextDevice().
I think EDK2 driver to restart cleanly for the next driver coming in.

Please share your thoughts in enhancing the logic for
AtaPassThruGetNextPort()/AtaPassThruGetNextDevice()

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4033] CryptoPkg: Add Hkdf SHA384 support

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4033

qi1.zhang@... <qi1.zhang@...> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|IN_PROGRESS |RESOLVED
Resolution|--- |FIXED

--- Comment #2 from qi1.zhang@... <qi1.zhang@...> ---
Merged https://github.com/tianocore/edk2/pull/3387

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4025] CryptoPkg: Add HMAC-SHA384 cipher support

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4025

qi1.zhang@... <qi1.zhang@...> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|CONFIRMED |RESOLVED
Resolution|--- |FIXED

--- Comment #2 from qi1.zhang@... <qi1.zhang@...> ---
Merged in PR:
https://github.com/tianocore/edk2/pull/3385

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4072] Code First / External ECR – RISC-V: Add Apic structure in MADT

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4072

--- Comment #2 from Sunil V L <sunilvl@...> ---
This ECR is approved as is by ASWG (no changes suggested)

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4072] Code First / External ECR – RISC-V: Add Apic structure in MADT

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4072

--- Comment #1 from Sunil V L <sunilvl@...> ---
Created attachment 1028
--> https://bugzilla.tianocore.org/attachment.cgi?id=1028&action=edit
ACPI_MADT_RINTC_ECR

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4073] Code First / External ECR – RISC-V: Add RHCT Table

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4073

--- Comment #3 from Sunil V L <sunilvl@...> ---
This ECR is approved by ASWG as is (without any changes suggested)

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4075] CryptoPkg: RsaGetPrivateKeyFromPem() crashed in Libfuzzer test

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4075

yi1.li@... <yi1.li@...> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |IN_PROGRESS
Ever confirmed|0 |1

--- Comment #4 from yi1.li@... <yi1.li@...> ---
Patch sent.
https://edk2.groups.io/g/devel/message/94119

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4073] Code First / External ECR – RISC-V: Add RHCT Table

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4073

Sunil V L <sunilvl@...> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #1025|0 |1
is obsolete| |

--- Comment #2 from Sunil V L <sunilvl@...> ---
Created attachment 1027
--> https://bugzilla.tianocore.org/attachment.cgi?id=1027&action=edit
RHCT_ECR

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4075] CryptoPkg: RsaGetPrivateKeyFromPem() crashed in Libfuzzer test

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4075

--- Comment #3 from yi1.li@... <yi1.li@...> ---
There are two solution choice:
1. Add true implementation to OpensslLib, but this will increase the 8kb
DXE driver binary size.
https://github.com/tianocore/edk2/pull/3361
2. Fake BIO_snprintf() need to return error status -1.
https://github.com/tianocore/edk2/pull/3360
Solution 1 need to turn on SSE when use GCC, it will be huge impact.

Recommend solution2.

--
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 4077] Remove enforcement of final GoIdle transition for CRB TPM commands.

bugzilla-daemon@...
 

https://bugzilla.tianocore.org/show_bug.cgi?id=4077

Rodrigo Gonzalez del Cueto <rodrigo.gonzalez.del.cueto@...> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|CONFIRMED |IN_PROGRESS

--- Comment #2 from Rodrigo Gonzalez del Cueto <rodrigo.gonzalez.del.cueto@...> ---
Proposed patch: https://edk2.groups.io/g/devel/message/94113

--
You are receiving this mail because:
You are on the CC list for the bug.

1 - 20 of 18941