[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/4] pci-arbiter: Fix a -Wstringop-truncation warning
From: |
Samuel Thibault |
Subject: |
Re: [PATCH 3/4] pci-arbiter: Fix a -Wstringop-truncation warning |
Date: |
Sun, 3 Nov 2019 17:20:17 +0100 |
User-agent: |
NeoMutt/20170609 (1.8.3) |
Hello,
Joan Lledó via Bug reports for the GNU Hurd, le dim. 03 nov. 2019 10:38:28
+0100, a ecrit:
> * pci-arbiter/pcifs.c:
> * create_dir_entry:
> Limit to NAME_SIZE-1 when calling strncpy().
> Finish entry->name with '\0'.
> * create_fs_tree:
> memset() to 0 the directory entry.
> Limit to NAME_SIZE-1 all calls to
> snprintf() and strncpy().
Applied, thanks!
> @@ -206,7 +208,7 @@ create_fs_tree (struct pcifs * fs)
> e_stat = list->stat;
> e_stat.st_mode &= ~S_IROOT; /* Remove the root mode */
> memset (entry_name, 0, NAME_SIZE);
> - snprintf (entry_name, NAME_SIZE, "%04x", device->domain);
> + snprintf (entry_name, NAME_SIZE - 1, "%04x", device->domain);
Perhaps replace the whole memset with just setting
entry_name[NAME_SIZE-1] = 0
? and ditto below.
Samuel
- Re: [PATCH] pci-arbiter: Remove embedded pciaccess code, Joan Lledó, 2019/11/03
- Re: [PATCH] pci-arbiter: Remove embedded pciaccess code, Joan Lledó, 2019/11/03
- [PATCH 2/4] pci-arbiter: Call libpciaccess cleanup on shutdown, Joan Lledó, 2019/11/03
- [PATCH 4/4] pci-arbiter: Fix warning on passing incompatible pointer type, Joan Lledó, 2019/11/03
- [PATCH 3/4] pci-arbiter: Fix a -Wstringop-truncation warning, Joan Lledó, 2019/11/03
- Re: [PATCH 3/4] pci-arbiter: Fix a -Wstringop-truncation warning,
Samuel Thibault <=
- Re: [PATCH 3/4] pci-arbiter: Fix a -Wstringop-truncation warning, James Clarke, 2019/11/03
- Re: [PATCH 3/4] pci-arbiter: Fix a -Wstringop-truncation warning, Samuel Thibault, 2019/11/03
- Re: [PATCH 3/4] pci-arbiter: Fix a -Wstringop-truncation warning, Joan Lledó, 2019/11/09
- [PATCH] pci-arbiter: Remove spurious memset(), Joan Lledó, 2019/11/09
- Re: [PATCH] pci-arbiter: Remove spurious memset(), Samuel Thibault, 2019/11/09
[PATCH 1/4] pci-arbiter: Remove embedded pciaccess code, Joan Lledó, 2019/11/03
Re: [PATCH] pci-arbiter: Remove embedded pciaccess code, Samuel Thibault, 2019/11/03