|  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #69222 Support PKCS#7 padding
Submitted: 2015-03-11 15:23 UTC Modified: 2017-01-10 10:15 UTC
From: dan at zaph dot com Assigned: leigh (profile)
Status: Not a bug Package: mcrypt related
PHP Version: 5.6.6 OS: all?
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
Solve the problem:
44 + 25 = ?
Subscribe to this entry?

 [2015-03-11 15:23 UTC] dan at zaph dot com
PKCS#7 is the industry standard for encryption padding. mcrypt does not support it, instead uses a no-standard null padding which does not work for binary data that might have a trailing 0 byte.

This default non-standard padding causes substantial interoperability problems. Is that what the php developers want?

Proposed solution: Add PKCS#7 padding option or remove mcrypt from distribution.

Or perhaps you guys are just lazy open source jerks that want to create interoperability problems for php users. OK, that made you mad, just like you make mcrypt used mad. Nice.


Add a Patch

Pull Requests

Add a Pull Request


AllCommentsChangesGit/SVN commitsRelated reports
 [2016-12-14 23:30 UTC]
-Status: Open +Status: Closed -Assigned To: +Assigned To: leigh
 [2016-12-14 23:30 UTC]
Closing because I'm such a lazy open source jerk I can't be bothered to tell the poster this isn't the place for libmcrypt feature requests.
 [2017-01-10 10:15 UTC]
-Status: Closed +Status: Not a bug
PHP Copyright © 2001-2022 The PHP Group
All rights reserved.
Last updated: Fri Dec 09 20:05:53 2022 UTC