diff options
| author | Günther Deschner <gd@samba.org> | 2018-05-08 13:46:11 +0200 |
|---|---|---|
| committer | Andreas Schneider <asn@cryptomilk.org> | 2018-05-17 17:30:09 +0200 |
| commit | 6326b3415f3e225aafd5912d0965c80abcd7b22c (patch) | |
| tree | 1b8e364e418f7e1e9351a83a22281c873c3676ab /source3/printing/printing.c | |
| parent | b24d4eb7afad82afc3a9bab65e1d799edc4b5172 (diff) | |
| download | samba-6326b3415f3e225aafd5912d0965c80abcd7b22c.tar.gz | |
s3-printing: fix format-truncation in print_queue_update()
../source3/printing/printing.c: In function ‘print_queue_update’:
../source3/printing/printing.c:1809:42: error: ‘%s’ directive output may be truncated writing up to 255 bytes into a region of size 244 [-Werror=format-truncation=]
snprintf(key, sizeof(key), "MSG_PENDING/%s", sharename);
^~ ~~~~~~~~~
../source3/printing/printing.c:1809:2: note: ‘snprintf’ output between 13 and 268 bytes into a destination of size 256
snprintf(key, sizeof(key), "MSG_PENDING/%s", sharename);
BUG: https://bugzilla.samba.org/show_bug.cgi?id=13437
Guenther
Signed-off-by: Guenther Deschner <gd@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Diffstat (limited to 'source3/printing/printing.c')
| -rw-r--r-- | source3/printing/printing.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/printing/printing.c b/source3/printing/printing.c index d8124686302..0acbb7e28ce 100644 --- a/source3/printing/printing.c +++ b/source3/printing/printing.c @@ -1700,7 +1700,7 @@ extern pid_t background_lpq_updater_pid; static void print_queue_update(struct messaging_context *msg_ctx, int snum, bool force) { - fstring key; + char key[268]; fstring sharename; char *lpqcommand = NULL; char *lprmcommand = NULL; |