[CRIU] [PATCH 2/2] compel: Add missing 32s support

Dmitry Safonov dsafonov at virtuozzo.com
Tue Apr 19 04:25:00 PDT 2016


On 04/19/2016 11:07 AM, Cyrill Gorcunov wrote:
> https://github.com/xemul/criu/issues/148#issuecomment-211635122
Oh, sorry for that, I should have included this in my compat-patches:
https://github.com/0x7f454c46/criu/commit/a688402e2d3c659ebd2ee3b4cea3c2d1d10712de
But I got that only after patches in the second part of compat series -
so planned to keep it in the second part.

I'm fine with yours version,
Reviewed-by: Dmitry Safonov <dsafonov at virtuozzo.com>

>
> Signed-off-by: Cyrill Gorcunov <gorcunov at openvz.org>
> ---
>   compel/src/elf.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/compel/src/elf.c b/compel/src/elf.c
> index 00f8c4926276..bd651770b58d 100644
> --- a/compel/src/elf.c
> +++ b/compel/src/elf.c
> @@ -415,6 +415,7 @@ int handle_elf(void *mem, size_t size)
>   
>   #ifdef ELF_X86_64
>   			case R_X86_64_32: /* Symbol + Addend (4 bytes) */
> +			case R_X86_64_32S: /* Symbol + Addend (4 bytes) */
>   				pr_debug("\t\t\t\tR_X86_64_32       at 0x%-4lx val 0x%x\n", place, value32);
>   				pr_out("	{ .offset = 0x%-8x, .type = PIEGEN_TYPE_INT, "
>   				       ".addend = %-8d, .value = 0x%-16x, }, /* R_X86_64_32 */\n",


-- 
Regards,
Dmitry Safonov



More information about the CRIU mailing list