hrev51959 adds 1 changeset to branch 'master'
old head: 172c944a5b58660ba40bc685faeb9f95733a6150
new head: fe3a28958a2483709832ce77bbe5b02c28602f29
overview:
https://git.haiku-os.org/haiku/log/?qt=range&q=fe3a28958a24+%5E172c944a5b58
----------------------------------------------------------------------------
fe3a28958a24: Appearance: Use BString::SetToFormat instead of sprintf to create
hex color.
Fixes a GCC7 Werror about (potential) format overflow.
[ Augustin Cavalier <waddlesplash@xxxxxxxxx> ]
----------------------------------------------------------------------------
Revision: hrev51959
Commit: fe3a28958a2483709832ce77bbe5b02c28602f29
URL: https://git.haiku-os.org/haiku/commit/?id=fe3a28958a24
Author: Augustin Cavalier <waddlesplash@xxxxxxxxx>
Date: Sun May 20 21:26:04 2018 UTC
----------------------------------------------------------------------------
1 file changed, 4 insertions(+), 3 deletions(-)
src/preferences/appearance/ColorPreview.cpp | 7 ++++---
----------------------------------------------------------------------------
diff --git a/src/preferences/appearance/ColorPreview.cpp
b/src/preferences/appearance/ColorPreview.cpp
index 3b522d52f6..94bfbf283f 100644
--- a/src/preferences/appearance/ColorPreview.cpp
+++ b/src/preferences/appearance/ColorPreview.cpp
@@ -17,6 +17,7 @@
#include <Bitmap.h>
#include <Message.h>
#include <MessageRunner.h>
+#include <String.h>
#include <View.h>
#include <Window.h>
@@ -227,11 +228,11 @@ ColorPreview::SetMode(bool rectangle)
void
ColorPreview::_DragColor(BPoint where)
{
- char hexString[7];
- sprintf(hexString, "#%.2X%.2X%.2X", fColor.red, fColor.green,
fColor.blue);
+ BString hexStr;
+ hexStr.SetToFormat("#%.2X%.2X%.2X", fColor.red, fColor.green,
fColor.blue);
BMessage message(B_PASTE);
- message.AddData("text/plain", B_MIME_TYPE, &hexString,
sizeof(hexString));
+ message.AddData("text/plain", B_MIME_TYPE, hexStr.String(),
hexStr.Length());
message.AddData("RGBColor", B_RGB_COLOR_TYPE, &fColor, sizeof(fColor));
BRect rect(0.0f, 0.0f, 20.0f, 20.0f);