RESOLVED FIXED 201903
Remove alternate declaration of CCECCryptorImportKey
https://bugs.webkit.org/show_bug.cgi?id=201903
Summary Remove alternate declaration of CCECCryptorImportKey
Keith Rollin
Reported 2019-09-17 22:01:12 PDT
In macOS 10.15 and iOS 13.0 SDKs, the keyPackage parameter of CCECCryptorImportKey() was changed from "void*" to "const void*". This change was accounted for in Bug 195754 by making the same change to the parallel declarations in CommonCryptoSPI.h. But since this header also needed to be used in macOS 10.14 contexts, the old declaration was also kept. But since this function is a "C" function, the types of parameters are not included in the function signature, and it doesn't really matter what parameter types are, so long as the right values are sent to the called function. This means that we don't need two versions of the function, one with a "const" parameter and one without. We only really need one. In this case, it makes sense to keep the more-restrictive one -- the with a "const void*" parameter. Therefore, remove the one taking just "void*".
Attachments
Patch (3.04 KB, patch)
2019-09-17 22:03 PDT, Keith Rollin
no flags
Radar WebKit Bug Importer
Comment 1 2019-09-17 22:01:28 PDT
Keith Rollin
Comment 2 2019-09-17 22:03:12 PDT
Alexey Proskuryakov
Comment 3 2019-09-18 09:45:20 PDT
Comment on attachment 379018 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=379018&action=review > Source/WebCore/PAL/ChangeLog:3 > + Remove alternate declaration of CCECCryptorImportKey This change is a local improvement, however this whole file isn’t following our normal convention. Normally, we redeclare functions in internal builds to catch changes in signatures, which can be more drastic than here.
WebKit Commit Bot
Comment 4 2019-09-18 12:40:00 PDT
Comment on attachment 379018 [details] Patch Clearing flags on attachment: 379018 Committed r250049: <https://trac.webkit.org/changeset/250049>
WebKit Commit Bot
Comment 5 2019-09-18 12:40:02 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.