HI Sam, I think this qualifies as obvious (it’s on my list, but I did not get to it yet, so go ahead).
Iain > On 2 Dec 2023, at 05:30, Sam James <s...@gentoo.org> wrote: > > From: Iain Sandoe <i...@sandoe.co.uk> > > r12-3005-g220c410162ebece4f missed a cast for the set_32 call. > Fixed thus. > > Signed-off-by: Iain Sandoe <i...@sandoe.co.uk> > Signed-off-by: Sam James <s...@gentoo.org> > > libiberty/ChangeLog: > PR other/112823 > * simple-object-mach-o.c (simple_object_mach_o_write_segment): > Cast the first argument to set_32 as needed. > > (cherry picked from commit 38757aa88735ab2e511bc428e2407a5a5e9fa0be) > --- > libiberty/simple-object-mach-o.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libiberty/simple-object-mach-o.c > b/libiberty/simple-object-mach-o.c > index 72b69d19c216..a8869e7c6395 100644 > --- a/libiberty/simple-object-mach-o.c > +++ b/libiberty/simple-object-mach-o.c > @@ -1228,7 +1228,7 @@ simple_object_mach_o_write_segment (simple_object_write > *sobj, int descriptor, > /* Swap the indices, if required. */ > > for (i = 0; i < (nsects_in * 4); ++i) > - set_32 (&index[i], index[i]); > + set_32 ((unsigned char *) &index[i], index[i]); > > sechdr_offset += sechdrsize; > > -- > 2.43.0 >