,
--
Mounir IDRASSI
IDRIX
http://www.idrix.fr
to install the consumer preview on my VirtualBox 4.1.8 the
same way as the developer preview (2G of RAM, 40G of disk, 128M Video
Memory, Use host I/O cache checked on the storage section).
I can run your test if you want.
Cheers,
--
Mounir IDRASSI
IDRIX
http://www.idrix.fr
languish for many years more!
Cheers,
--
Mounir IDRASSI
IDRIX
http://www.idrix.fr
provide or compiling their own using the original patch.
Cheers,
--
Mounir IDRASSI
IDRIX
http://www.idrix.fr
Hi,
You're right: I have done a small test and MSDN is indeed wrong on the
definition of this function. Sorry for my erroneous patch.
--
Mounir IDRASSI
IDRIX
http://www.idrix.fr
On Wed, August 27, 2008 4:43 am, Dmitry Timoshkov wrote:
Mounir IDRASSI [EMAIL PROTECTED] wrote:
This patch
on it. The source from my side has
evolved a little bit but it remains close to the original posting. I hope
there will room for collaboration of this to provide a maximum number of
Wine users with smart cards functionalities.
Cheers,
--
Mounir IDRASSI
IDRIX
http://www.idrix.fr
szRSAName should be changed to static const WCHAR szRSAName[4][66] and
its value appended at the end with the new name of the provider for XP
instead of modifying the last field. This way, I'm sure the patch will be
accepted.
Vijay : can you submit a modified version of your patch.
Cheers,
Mounir
You forgot the attachement...
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Juan Lang wrote:
--Juan
Choose the right car based on your needs. Check out Yahoo
is straightforward: in the code of
RSAENH_CPAcquireContext, juste add the line case CRYPT_VERIFYCONTEXT |
CRYPT_NEWKEYSET: above the existing line case CRYPT_VERIFYCONTEXT:.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Maarten Lankhorst wrote:
With tests. I think
you please send the patching code along with the test to the
patches list?.
Thanks for pointing this behavior.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Maarten Lankhorst wrote:
Hello,
After retrying, I tried the same function arguments that msn messenger
that
CRYPT_VERIFYCONTEXT is reserved for non persistent operations which is
not the case when you create a new container.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Maarten Lankhorst wrote:
According to msdn a combination of flags are set, since I'm not sure
which combo's
Hi,
You also have to modify other files in order to integrate your stub
implementation to the wine build chain: Makefile.in, configure and
configure.ac in the wine root directory, Makefile.in in the dlls
directory.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
a 64bit host and
pcsclite is compiled on 64bit mode...but I think this is a more global
issue.
Am I missing something? Please advise.
Thanks.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Alexandre Julliard wrote:
I don't think the pcsc-lite library is guaranteed
of the whole implementation.
Concerning the BOOL return value, it's here for historical reasons. I'll
remove it.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://.idrix.fr
+BOOL InitializePCSCLite(void)
You don't use the return value:
@@ -42,9 +43,11 @@ BOOL WINAPI DllMain
The code is definitely cleaner with all your suggestions. I'm applying
them now.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
James wrote:
This would be a lot cleaner as:
if (!(liteSCardEstablishContext = (SCardEstablishContextPtr)
wine_dlsym
Hi Alexandre,
You are completely right...these variables are definitely useless!! I
don't remember when I coded that but it must have been very late at night!
Anyway, I'll purge them from the patch.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Alexandre
Hi Francois,
Thanks for your modifications. I'll review your patch and submit it if I
have no comments,
ِCheers,
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Francois Gouget wrote:
Hi,
Here's a new version of the patch that removes the casts as suggested
Hi,
Apart from Alexandre comment, I don't see any other possible changes.
So, if the first patch containing the API headers is OK, I can submit
the winscard stub patch.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Alexandre Julliard wrote:
Francois Gouget
complete
patch would include also a correction to this file.
Question: can a patch include corrections to different dlls? I can
submit it now.
Cheers
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
Alexandre Julliard wrote:
Mounir IDRASSI [EMAIL PROTECTED] writes:
This patch adds
a correct patch soon.
I analogize for my misunderstanding.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Alexandre Julliard wrote:
Mounir IDRASSI [EMAIL PROTECTED] writes:
Like I said, the test currently passes on Windows, so you can't simply
change it. Tests
Actually, the only application I know that uses this expects only
CRYPT_SEC_DESCR. So, as you advised, I'll rewrite the patch to remove
the version check.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Alexandre Julliard wrote:
Mounir IDRASSI [EMAIL PROTECTED
? This is the only case for
the test to fail.
Cheers,
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Juan Lang wrote:
Hi Mounir,
+case PP_KEYSTORAGE:
+dwTemp = CRYPT_SEC_DESCR;
This looks fine..
+result = CryptGetProvParam(prov
Hi Juan,
Thanks for your explanations. I'm resubmitting the patch.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Juan Lang wrote:
Yes, certainly. See e.g. recent patches from Paul Vriens to get the tests
succeeding on Win9x.
--Juan
Hi,
In the tests of the crypt32 dll, a test was written in a way that
accommodates the bug we have corrected previously in the patch for
rsaenh dll. The attached small patch corrects the value expected from
this test to be consistent with the correct output.
Cheers,
Mounir IDRASSI
IDRIX
is a patch correcting this bug and adding a
test that demonstrate it.
Cheers,
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
From 8da78d912c0a709a641ae4082ee6df270f644a08 Mon Sep 17 00:00:00 2001
From: Mounir IDRASSI [EMAIL PROTECTED]
Date: Mon, 7 May 2007 17:04:46
,
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Thanks for your reply. I'm rewriting the patches and sending them back.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Detlef Riekenberg wrote:
I have no Idea about the code, but the missing defines should
be a seperate patch.
Your idention looks broken
yes but I prefer sending patches first to the devel list for comments.
Once no major issue is raised, I send it to the patches list. I think
this helps prevent polluting the patches list.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Stefan Dösinger wrote:
Hi
I'll take you advice from now and I apologize for any annoyance caused
by my previous emails.
I would also thank all the people who took time to review our patches
and sent me their comments/remarks.
Regards,
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Hi,
Thanks Maarte for your comment. Please find attached the patch augmented
with some tests that illustrate what has been corrected.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Maarten Lankhorst wrote:
I don't see any tests added in this patch so it is hard
, PP_KEYSTORAGE.
In order to support the PP_KEYSTORAGE parameter, we added three defined
to wincrypt.h.
Cheers,
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
From a7240539a2e3c83291c7cf206d55f1e4e7e75803 Mon Sep 17 00:00:00 2001
From: Mounir IDRASSI [EMAIL
Sorry, my mistake... I'll definitely be more careful next time.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Dmitry Timoshkov wrote:
This is a notice more for Alexandre this time, but next time please
make sure to send the files with exec bits turned off (755
them if
they report an error. That's why we must implement them all. But I can
start with stubs in the spec file for the first patch.
I'll wait for your reply before submitting the first patch.
Thanks.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Eric Pouech
Hi,
Please find attached the patch for adding header files for the support
of smart cards through Winscard dll.
If it's OK, should I post it elsewhere?
Thanks in advance.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Eric Pouech wrote:
sorry, I didn't
with msvcrt source is that
winscard exported variables have a structure as a type, whereas msvcrt
exported variables are pointers.
Cheers,
Mounir IDRASSI
IDRIX – Cryptography and IT Security Experts
http://www.idrix.fr
Eric Pouech wrote:
from a cursory look, it looks a very goot start !!!
a couple
and
WideCharToMultiString don't understand multi-strings ( MSDN say nothing
about that).
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Eric Pouech wrote:
you have to ensure that binary compat is present for both pcsc-lite
and MS APIs
if not, then you'll have to convert back
oups ...sorry...It was my fault! You are as absolutely right.
Mounir IDRASSI
IDRIX - Cryptography and IT Security Experts
http://www.idrix.fr
Eric Pouech wrote:
they should work across the whole buffer
are you sure you don't pass -1 as the length for the input buffer ?
A+
. However, we would like
to known the reason behind the lack of winscard support. Is it simply because
no one needed it before or maybe there is an issue with the pcsc-lite license?
Thanks in advance for your information.
Cheers,
Mounir IDRASSI
IDRIX
http://www.idrix.fr
are not really experts in autoconf and
libtool scripts, we started from a pre-configured source tree and adapted the
files of the cryptdll library (the simplest one). So we will definitely need
the help of the community to integrate our source seamlessly into the wine
source tree.
Mounir IDRASSI
IDRIX
39 matches
Mail list logo