|
 |
dacebc |
From 6ed2f7d1a379f69cca102e8166d20eb5ed38652b Mon Sep 17 00:00:00 2001
|
|
|
dacebc |
From: akallabeth <akallabeth@posteo.net>
|
|
|
dacebc |
Date: Fri, 22 Apr 2022 16:27:21 +0200
|
|
|
dacebc |
Subject: [PATCH] Fixed format string for Stream_CheckAndLogRequiredLength
|
|
|
dacebc |
|
|
|
dacebc |
__LINE__ requires %d and not %PRIuz
|
|
|
dacebc |
|
|
|
dacebc |
(cherry picked from commit 74c1a006e940308b0653427d25a87ea5a24cb573)
|
|
|
dacebc |
---
|
|
|
dacebc |
winpr/include/winpr/stream.h | 14 ++++++++
|
|
|
dacebc |
winpr/libwinpr/utils/stream.c | 65 +++++++++++++++++++++++++++++++++++
|
|
|
dacebc |
2 files changed, 79 insertions(+)
|
|
|
dacebc |
|
|
|
dacebc |
diff --git a/winpr/include/winpr/stream.h b/winpr/include/winpr/stream.h
|
|
|
dacebc |
index f351eaa15..ed637f034 100644
|
|
|
dacebc |
--- a/winpr/include/winpr/stream.h
|
|
|
dacebc |
+++ b/winpr/include/winpr/stream.h
|
|
|
dacebc |
@@ -27,6 +27,8 @@
|
|
|
dacebc |
#include <winpr/wtypes.h>
|
|
|
dacebc |
#include <winpr/endian.h>
|
|
|
dacebc |
#include <winpr/synch.h>
|
|
|
dacebc |
+#include <winpr/wlog.h>
|
|
|
dacebc |
+#include <winpr/debug.h>
|
|
|
dacebc |
|
|
|
dacebc |
#ifdef __cplusplus
|
|
|
dacebc |
extern "C"
|
|
|
dacebc |
@@ -56,6 +57,19 @@ extern "C"
|
|
|
dacebc |
WINPR_API void Stream_StaticInit(wStream* s, BYTE* buffer, size_t size);
|
|
|
dacebc |
WINPR_API void Stream_Free(wStream* s, BOOL bFreeBuffer);
|
|
|
dacebc |
|
|
|
dacebc |
+#define Stream_CheckAndLogRequiredLength(tag, s, len) \
|
|
|
dacebc |
+ Stream_CheckAndLogRequiredLengthEx(tag, WLOG_WARN, s, len, "%s(%s:%d)", __FUNCTION__, \
|
|
|
dacebc |
+ __FILE__, __LINE__)
|
|
|
dacebc |
+ WINPR_API BOOL Stream_CheckAndLogRequiredLengthEx(const char* tag, DWORD level, wStream* s,
|
|
|
dacebc |
+ UINT64 len, const char* fmt, ...);
|
|
|
dacebc |
+ WINPR_API BOOL Stream_CheckAndLogRequiredLengthExVa(const char* tag, DWORD level, wStream* s,
|
|
|
dacebc |
+ UINT64 len, const char* fmt, va_list args);
|
|
|
dacebc |
+ WINPR_API BOOL Stream_CheckAndLogRequiredLengthWLogEx(wLog* log, DWORD level, wStream* s,
|
|
|
dacebc |
+ UINT64 len, const char* fmt, ...);
|
|
|
dacebc |
+ WINPR_API BOOL Stream_CheckAndLogRequiredLengthWLogExVa(wLog* log, DWORD level, wStream* s,
|
|
|
dacebc |
+ UINT64 len, const char* fmt,
|
|
|
dacebc |
+ va_list args);
|
|
|
dacebc |
+
|
|
|
dacebc |
static INLINE void Stream_Seek(wStream* s, size_t _offset)
|
|
|
dacebc |
{
|
|
|
dacebc |
s->pointer += (_offset);
|
|
|
dacebc |
diff --git a/winpr/libwinpr/utils/stream.c b/winpr/libwinpr/utils/stream.c
|
|
|
dacebc |
index 1271981b7..cc119c771 100644
|
|
|
dacebc |
--- a/winpr/libwinpr/utils/stream.c
|
|
|
dacebc |
+++ b/winpr/libwinpr/utils/stream.c
|
|
|
dacebc |
@@ -132,3 +132,68 @@ void Stream_Free(wStream* s, BOOL bFreeBuffer)
|
|
|
dacebc |
free(s);
|
|
|
dacebc |
}
|
|
|
dacebc |
}
|
|
|
dacebc |
+
|
|
|
dacebc |
+BOOL Stream_CheckAndLogRequiredLengthEx(const char* tag, DWORD level, wStream* s, UINT64 len,
|
|
|
dacebc |
+ const char* fmt, ...)
|
|
|
dacebc |
+{
|
|
|
dacebc |
+ const size_t actual = Stream_GetRemainingLength(s);
|
|
|
dacebc |
+
|
|
|
dacebc |
+ if (actual < len)
|
|
|
dacebc |
+ {
|
|
|
dacebc |
+ va_list args;
|
|
|
dacebc |
+
|
|
|
dacebc |
+ va_start(args, fmt);
|
|
|
dacebc |
+ Stream_CheckAndLogRequiredLengthExVa(tag, level, s, len, fmt, args);
|
|
|
dacebc |
+ va_end(args);
|
|
|
dacebc |
+
|
|
|
dacebc |
+ return FALSE;
|
|
|
dacebc |
+ }
|
|
|
dacebc |
+ return TRUE;
|
|
|
dacebc |
+}
|
|
|
dacebc |
+
|
|
|
dacebc |
+BOOL Stream_CheckAndLogRequiredLengthExVa(const char* tag, DWORD level, wStream* s, UINT64 len,
|
|
|
dacebc |
+ const char* fmt, va_list args)
|
|
|
dacebc |
+{
|
|
|
dacebc |
+ const size_t actual = Stream_GetRemainingLength(s);
|
|
|
dacebc |
+
|
|
|
dacebc |
+ if (actual < len)
|
|
|
dacebc |
+ return Stream_CheckAndLogRequiredLengthWLogExVa(WLog_Get(tag), level, s, len, fmt, args);
|
|
|
dacebc |
+ return TRUE;
|
|
|
dacebc |
+}
|
|
|
dacebc |
+
|
|
|
dacebc |
+BOOL Stream_CheckAndLogRequiredLengthWLogEx(wLog* log, DWORD level, wStream* s, UINT64 len,
|
|
|
dacebc |
+ const char* fmt, ...)
|
|
|
dacebc |
+{
|
|
|
dacebc |
+ const size_t actual = Stream_GetRemainingLength(s);
|
|
|
dacebc |
+
|
|
|
dacebc |
+ if (actual < len)
|
|
|
dacebc |
+ {
|
|
|
dacebc |
+ va_list args;
|
|
|
dacebc |
+
|
|
|
dacebc |
+ va_start(args, fmt);
|
|
|
dacebc |
+ Stream_CheckAndLogRequiredLengthWLogExVa(log, level, s, len, fmt, args);
|
|
|
dacebc |
+ va_end(args);
|
|
|
dacebc |
+
|
|
|
dacebc |
+ return FALSE;
|
|
|
dacebc |
+ }
|
|
|
dacebc |
+ return TRUE;
|
|
|
dacebc |
+}
|
|
|
dacebc |
+
|
|
|
dacebc |
+BOOL Stream_CheckAndLogRequiredLengthWLogExVa(wLog* log, DWORD level, wStream* s, UINT64 len,
|
|
|
dacebc |
+ const char* fmt, va_list args)
|
|
|
dacebc |
+{
|
|
|
dacebc |
+ const size_t actual = Stream_GetRemainingLength(s);
|
|
|
dacebc |
+
|
|
|
dacebc |
+ if (actual < len)
|
|
|
dacebc |
+ {
|
|
|
dacebc |
+ char prefix[1024] = { 0 };
|
|
|
dacebc |
+
|
|
|
dacebc |
+ vsnprintf(prefix, sizeof(prefix), fmt, args);
|
|
|
dacebc |
+
|
|
|
dacebc |
+ WLog_Print(log, level, "[%s] invalid length, got %" PRIuz ", require at least %" PRIu64,
|
|
|
dacebc |
+ prefix, actual, len);
|
|
|
dacebc |
+ winpr_log_backtrace_ex(log, level, 20);
|
|
|
dacebc |
+ return FALSE;
|
|
|
dacebc |
+ }
|
|
|
dacebc |
+ return TRUE;
|
|
|
dacebc |
+}
|
|
|
dacebc |
--
|
|
|
dacebc |
2.38.1
|
|
|
dacebc |
|