From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751017AbeAOWhE (ORCPT + 1 other); Mon, 15 Jan 2018 17:37:04 -0500 Received: from mx0b-001b2d01.pphosted.com ([148.163.158.5]:36302 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750914AbeAOWhB (ORCPT ); Mon, 15 Jan 2018 17:37:01 -0500 Date: Mon, 15 Jan 2018 23:36:52 +0100 From: Heiko Carstens To: Eugene Syromiatnikov Cc: linux-s390@vger.kernel.org, Martin Schwidefsky , Paul Gortmaker , Philippe Ombredanne , Greg Kroah-Hartman , Kate Stewart , Thomas Gleixner , linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] s390: fix handling of -1 in set{,fs}[gu]id16 syscalls References: <20180115115242.GA15131@asgard.redhat.com> <20180115193817.GA8145@asgard.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180115193817.GA8145@asgard.redhat.com> X-TM-AS-GCONF: 00 x-cbid: 18011522-0020-0000-0000-000003EAF2E6 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 18011522-0021-0000-0000-0000427D2764 Message-Id: <20180115223652.GB3054@osiris> X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2018-01-15_09:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 malwarescore=0 suspectscore=1 phishscore=0 bulkscore=0 spamscore=0 clxscore=1015 lowpriorityscore=0 impostorscore=0 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1709140000 definitions=main-1801150316 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Mon, Jan 15, 2018 at 08:38:17PM +0100, Eugene Syromiatnikov wrote: > For some reason, the implementation of some 16-bit ID system calls > (namely, setuid16/setgid16 and setfsuid16/setfsgid16) used type cast > instead of low2highgid/low2highuid macros for converting [GU]IDs, which > led to incorrect handling of value of -1 (which ought to be considered > invalid). > > Discovered by strace test suite. > > Cc: stable@vger.kernel.org > Signed-off-by: Eugene Syromiatnikov > --- > arch/s390/kernel/compat_linux.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) Applied, thanks!