Skip to content

Conversation

@xhanulik
Copy link
Member

The integration of Packit for testing the build on Fedora was already discussed in #2528.
Since there was a problem with installing the packit-as-a-service app in GH actions for OpenSC, this PR adds separate actions for creating SRPM and building RPM with Packit.

As the local build command that Packit provides does not install BuildRequires from the spec file nor provide a command for that, there is also a separate script providing the installation of the dependencies.

dengert added a commit to dengert/OpenSC that referenced this pull request Oct 26, 2022
Include a fix for OpenSC#2625

OpenSC definitions of ATRs have been lax in "sc_atr_table" entries by allowing
1 bits in the ATR that need to be 0 bits when used with Windows compare
Do the equivelent reduction of the table ATR done in card.c by "tbin[s] = (tbin[s] & mbin[s]);"
before adding to registry.

This will then allow lax definitions of ATRs to be copied to customactions.cpp
and fixed before adding to registry.

 Changes to be committed:
	modified:   win32/customactions.cpp
dengert added a commit to dengert/OpenSC that referenced this pull request Oct 26, 2022
Include a fix for OpenSC#2625

OpenSC definitions of ATRs have been lax in "sc_atr_table" entries by allowing
1 bits in the ATR that need to be 0 bits when used with Windows compare
Do the equivelent reduction of the table ATR done in card.c by "tbin[s] = (tbin[s] & mbin[s]);"
before adding to registry.

This will then allow lax definitions of ATRs to be copied to customactions.cpp
and fixed before adding to registry.

 Changes to be committed:
	modified:   win32/customactions.cpp
dengert added a commit to dengert/OpenSC that referenced this pull request Oct 26, 2022
Include a fix for OpenSC#2625

OpenSC definitions of ATRs have been lax in "sc_atr_table" entries by allowing
1 bits in the ATR that need to be 0 bits when used with Windows compare
Do the equivalent reduction of the table ATR done in card.c by "tbin[s] = (tbin[s] & mbin[s]);"
before adding to registry.

This will then allow lax definitions of ATRs to be copied to customactions.cpp
and fixed before adding to registry.

 Changes to be committed:
	modified:   win32/customactions.cpp
dengert added a commit to dengert/OpenSC that referenced this pull request Oct 27, 2022
Include a fix for OpenSC#2625

OpenSC definitions of ATRs have been lax in "sc_atr_table" entries by allowing
1 bits in the ATR that need to be 0 bits when used with Windows compare
Do the equivalent reduction of the table ATR done in card.c by "tbin[s] = (tbin[s] & mbin[s]);"
before adding to registry.

This will then allow lax definitions of ATRs to be copied to customactions.cpp
and fixed before adding to registry.

 Changes to be committed:
	modified:   win32/customactions.cpp
@Jakuje Jakuje merged commit 782e93e into OpenSC:master Oct 30, 2022
@xhanulik xhanulik deleted the packit-rpm branch October 31, 2022 13:31
Jakuje pushed a commit that referenced this pull request Nov 9, 2022
Include a fix for #2625

OpenSC definitions of ATRs have been lax in "sc_atr_table" entries by allowing
1 bits in the ATR that need to be 0 bits when used with Windows compare
Do the equivalent reduction of the table ATR done in card.c by "tbin[s] = (tbin[s] & mbin[s]);"
before adding to registry.

This will then allow lax definitions of ATRs to be copied to customactions.cpp
and fixed before adding to registry.

 Changes to be committed:
	modified:   win32/customactions.cpp
AlexandreGonzalo pushed a commit to AlexandreGonzalo/OpenSC that referenced this pull request Nov 14, 2022
Include a fix for OpenSC#2625

OpenSC definitions of ATRs have been lax in "sc_atr_table" entries by allowing
1 bits in the ATR that need to be 0 bits when used with Windows compare
Do the equivalent reduction of the table ATR done in card.c by "tbin[s] = (tbin[s] & mbin[s]);"
before adding to registry.

This will then allow lax definitions of ATRs to be copied to customactions.cpp
and fixed before adding to registry.

 Changes to be committed:
	modified:   win32/customactions.cpp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants