From 6cf17e76b0bae7455581f0002fb79ffc2f1858a1 Mon Sep 17 00:00:00 2001 From: Christopher Hoffman Date: Tue, 3 Jun 2025 15:13:16 +0000 Subject: [PATCH] Address misc comments Signed-off-by: Christopher Hoffman --- src/client/FSCrypt.cc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/client/FSCrypt.cc b/src/client/FSCrypt.cc index 32269b0afb9..d3f162c8af2 100644 --- a/src/client/FSCrypt.cc +++ b/src/client/FSCrypt.cc @@ -42,12 +42,13 @@ using ceph::crypto::HMACSHA512; -/* FIXME: this was copy pasted from common/armor.c with slight modification +/* FIXME: Use boost or similar library to roll your own + * FIXME: this was copy pasted from common/armor.c with slight modification * as needed to use alternative translation table. Code can and should be * combined, but need to make sure we do it in a way that doesn't hurt * compiler optimizations in the general case. * Also relaxed decoding to make it compatible with the kernel client */ -static const char *pem_key = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+,"; +static const char pem_key[] = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+,"; static int encode_bits(int c) { -- 2.47.3