From: James Almer <jamrial@gmail.com>
To: ffmpeg-devel@ffmpeg.org
Subject: [FFmpeg-devel] [PATCH 6/6] avutil/tests/aes_ctr: also randomize the encryption key
Date: Thu, 3 Apr 2025 17:59:03 -0300
Message-ID: <20250403205903.1390-3-jamrial@gmail.com> (raw)
In-Reply-To: <20250401151637.2021-1-jamrial@gmail.com>
And not just the IV.
Signed-off-by: James Almer <jamrial@gmail.com>
---
libavutil/tests/aes_ctr.c | 25 ++++++++++++++++++++++---
1 file changed, 22 insertions(+), 3 deletions(-)
diff --git a/libavutil/tests/aes_ctr.c b/libavutil/tests/aes_ctr.c
index 6f21b8bda1..fad24014cc 100644
--- a/libavutil/tests/aes_ctr.c
+++ b/libavutil/tests/aes_ctr.c
@@ -18,6 +18,7 @@
#include <string.h>
+#include "libavutil/random_seed.h"
#include "libavutil/log.h"
#include "libavutil/mem_internal.h"
#include "libavutil/aes_ctr.h"
@@ -36,13 +37,20 @@ static const DECLARE_ALIGNED(8, uint8_t, fixed_iv)[] = {
0xde, 0xad, 0xbe, 0xef, 0xde, 0xad, 0xbe, 0xef
};
+static const DECLARE_ALIGNED(8, uint8_t, fixed_key)[] = {
+ 0x30, 0x31, 0x32, 0x33, 0x34, 0x35, 0x36, 0x37,
+ 0x38, 0x39, 0x61, 0x62, 0x63, 0x64, 0x65, 0x66
+};
+
+static DECLARE_ALIGNED(8, uint32_t, key)[2];
+
static DECLARE_ALIGNED(8, uint8_t, tmp)[20];
int main (void)
{
int ret = 1;
struct AVAESCTR *ae, *ad;
- const uint8_t *iv;
+ const uint8_t *iv, *k;
for (int i = 0; i < 2; i++) {
ae = av_aes_ctr_alloc();
@@ -51,10 +59,21 @@ int main (void)
if (!ae || !ad)
goto ERROR;
- if (av_aes_ctr_init(ae, (const uint8_t*)"0123456789abcdef") < 0)
+ if (i)
+ k = fixed_key;
+ else {
+ // Note: av_random_bytes() should be used in a real world scenario,
+ // but since that function can fail, av_get_random_seed() is used
+ // here for the purpose of this test, as its output is sufficient.
+ key[0] = av_get_random_seed();
+ key[1] = av_get_random_seed();
+ k = (uint8_t *)key;
+ }
+
+ if (av_aes_ctr_init(ae, k) < 0)
goto ERROR;
- if (av_aes_ctr_init(ad, (const uint8_t*)"0123456789abcdef") < 0)
+ if (av_aes_ctr_init(ad, k) < 0)
goto ERROR;
if (i)
--
2.49.0
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
prev parent reply other threads:[~2025-04-03 20:59 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-01 15:16 [FFmpeg-devel] [PATCH 1/2] avutil/aes_ctr: simplify and optimize av_aes_ctr_crypt() James Almer
2025-04-01 15:16 ` [FFmpeg-devel] [PATCH 2/2] avutil/aes_ctr: simplify incrementing the counter James Almer
2025-04-01 15:18 ` [FFmpeg-devel] [PATCH 1/2] avutil/aes_ctr: simplify and optimize av_aes_ctr_crypt() Nicolas George
2025-04-01 15:29 ` [FFmpeg-devel] [PATCH v2 " James Almer
2025-04-01 17:09 ` [FFmpeg-devel] [PATCH 3/3] avutil/tests/aes_ctr: test more than a single block worth of data James Almer
2025-04-03 20:59 ` [FFmpeg-devel] [PATCH 4/6] avutil/test/aes_ctr: also check the encrypted buffer James Almer
2025-04-03 20:59 ` [FFmpeg-devel] [PATCH 5/6] avutil/tests/aes_ctr: reindent after the previous commit James Almer
2025-04-03 20:59 ` James Almer [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250403205903.1390-3-jamrial@gmail.com \
--to=jamrial@gmail.com \
--cc=ffmpeg-devel@ffmpeg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git