On Thu, Oct 17, 2024 at 03:52:51PM +0800, Liu Jing wrote:
> There is no va_end after va_copy, just add it.
>
> Signed-off-by: Liu Jing
> Signed-off-by: Liu Jing
Thanks,
This is correct. However, it was fixed recently in the same way
by the following commit authored by one of your colleagues, Z
Hi Martin, thanks for the review !
On 10/19/24 01:57, Martin KaFai Lau wrote:
> On 10/16/24 11:35 AM, Alexis Lothoré (eBPF Foundation) wrote:
>> btf_skc_cls_ingress.c currently runs two subtests, and create a
>> dedicated network namespace for each, but never cleans up the created
>> namespace onc
> + strhdr = &info->sechdrs[info->index.str];
> + if (strhdr->sh_size > 0 && info->strtab[strhdr->sh_size - 1] != '\0') {
> + pr_err("module %s: string table isn't null terminated\n",
> +info->name ?: "(missing .modinfo section or name
> field)");
> +
On Sat, Oct 19, 2024 at 04:15:33PM +0200, Tobias Stoeckmann wrote:
> + if (sym[i].st_name >= strhdr->sh_size) {
Please note that this commit only makes sense being applied AFTER
the other patch sent, i.e. "module: .strtab must be null terminated"
because that patch modifies strhdr befo
On Mon, 2024-10-14 at 13:12 -0700, Jakub Kicinski wrote:
> On Mon, 14 Oct 2024 08:25:35 +0100 David Woodhouse wrote:
> > On Wed, 2024-10-09 at 17:32 -0700, Jakub Kicinski wrote:
> > > On Sun, 06 Oct 2024 08:17:58 +0100 David Woodhouse wrote:
> > > > +config PTP_1588_CLOCK_VMCLOCK
> > > > +
Hi Antonio,
kernel test robot noticed the following build warnings:
[auto build test WARNING on 6d858708d465669ba7de17e9c5691eb4019166e8]
url:
https://github.com/intel-lab-lkp/linux/commits/Antonio-Quartulli/netlink-add-NLA_POLICY_MAX_LEN-macro/20241016-092722
base: 6d858708d465669ba7de17e
Corrected the multiple and different typo errors in json files
- "diffferent" is corrected to "different".
- "muliple" and "miltiple" is corrected to "multiple".
Signed-off-by: Karan Sanghavi
---
tools/testing/selftests/tc-testing/tc-tests/filters/basic.json | 6 +++---
tools/testing/selftests
On Sat, Oct 19, 2024 at 11:52:21AM +, Karan Sanghavi wrote:
> Corrected the multiple and different typo errors in json files
>
> - "diffferent" is corrected to "different".
> - "muliple" and "miltiple" is corrected to "multiple".
>
> Signed-off-by: Karan Sanghavi
Thanks,
This addresses all
Use the new shared modules tree as we have more than one
modules maintainer now.
git://git.kernel.org/pub/scm/linux/kernel/git/modules/linux.git
Signed-off-by: Luis Chamberlain
---
Next/Trees | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Next/Trees b/Next/Trees
index 38812
On Wed, Oct 16, 2024 at 04:18:41PM +0200, Thorsten Leemhuis wrote:
> Switch away from using sha1 for module signing by default and use the
> more modern sha512 instead, which is what among others Arch, Fedora,
> RHEL, and Ubuntu are currently using for their kernels.
>
> Sha1 has not been consider
On 08/10/24 8:22 PM, Shivam Chaudhary wrote:
Corrected minor typo in tools/virtio/vringh_test.c:
- Fixed "retreives" to "retrieves"
Signed-off-by: Shivam Chaudhary
---
tools/virtio/vringh_test.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/virtio/vringh_test.c
On Donnerstag, 6. Juni 2024 21:01:36 MESZ Luca Weiss wrote:
> When the mailbox driver has not probed yet, the error message "failed to
> parse smd edge" is just going to confuse users, so improve the error
> prints a bit.
>
> Cover the last remaining exits from qcom_smd_parse_edge with proper
> er
On 10/19/24 02:30, Martin KaFai Lau wrote:
> On 10/16/24 11:35 AM, Alexis Lothoré (eBPF Foundation) wrote:
[...]
>> + switch (ip_mode) {
>> + case TEST_MODE_IPV4:
>> + sock_family = AF_INET;
>> + srv_addr = SERVER_ADDR_IPV4;
>> + addr = (struct sockaddr_storage *)&srv
The string table must be NUL-terminated, just like the section name table.
Signed-off-by: Tobias Stoeckmann
---
In order to create a proof of concept, which I can't get into a simple
script right now, it's easiest to move '.strtab' to the end of the module
file, write as many 'A' characters at th
It must be verified that the symbol name offsets point into the
string table, not outside of it.
Signed-off-by: Tobias Stoeckmann
---
Proof of Concept:
1. Create "poc.sh"
```
cat > poc.sh << EOF
#!/bin/sh
# Sets an illegal symbol name offset in supplied uncompressed module
# usage: ./poc file.k
On Thu, Oct 17, 2024 at 02:08:19PM +0200, Helge Deller wrote:
> Hi Luis,
>
> On 10/17/24 01:21, Luis Chamberlain wrote:
> > That sounds great. Yeah, the above would be great to test. A while ago
> > I wrote a new modules selftests in order to test possible improvements
> > on find_symbol() but I a
On Tue, Oct 15, 2024 at 11:16:34PM +, Matthew Maurer wrote:
> Split out from Extended MODVERSIONS Support [1]
>
> This series refactors module validation during loading to ensure that
> everything is checked on its way in. This is intended to make the code
> robust enough that we can more conf
On Thu, Oct 17, 2024 at 03:45:56PM -0300, Jason Gunthorpe wrote:
> > +struct iommufd_vdevice *
> > +__iommufd_vdevice_alloc(struct iommufd_ctx *ictx, size_t size)
> > +{
> > + struct iommufd_object *obj;
> > +
> > + if (WARN_ON(size < sizeof(struct iommufd_vdevice)))
> > + return ERR_
On Thu, Oct 17, 2024 at 03:52:30PM -0300, Jason Gunthorpe wrote:
> > + if (viommu->ops && viommu->ops->vdevice_alloc)
> > + vdev = viommu->ops->vdevice_alloc(viommu, idev->dev, virt_id);
> > + else
> > + vdev = __iommufd_vdevice_alloc(ucmd->ictx, sizeof(*vdev));
> > + if (
19 matches
Mail list logo
