Message ID | c4f64515-fec4-24dc-47a2-76b091718530@oracle.com |
---|---|
State | Superseded |
Headers |
Return-Path: <libc-alpha-bounces+patchwork=sourceware.org@sourceware.org> X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 5FBFD384F6FA for <patchwork@sourceware.org>; Wed, 23 Nov 2022 21:08:35 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 5FBFD384F6FA DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1669237715; bh=zoHR3jJsZ8KlzLSfenDdeofxmepyJJcfrZ4b1+Sc0Yg=; h=Date:To:Subject:List-Id:List-Unsubscribe:List-Archive:List-Post: List-Help:List-Subscribe:From:Reply-To:From; b=mPlzQ0TqCF8EUhYSgghOt0sUNQk7G0qKNwS7Y0/k5dhwwnaAdd+9Xy+ricu+Ohbso c02fo0XI87t0aE5E/VaQA8e+zfOpc1k71TNCgIDlyRtwNHInj4aup4ucvIo1yeG14d +01RDWREMQfDwJ/NgAuqj0vnftU+Iuwyl02cvTgI= X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from mx0b-00069f02.pphosted.com (mx0b-00069f02.pphosted.com [205.220.177.32]) by sourceware.org (Postfix) with ESMTPS id A9138384F6E5 for <libc-alpha@sourceware.org>; Wed, 23 Nov 2022 21:08:12 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org A9138384F6E5 Received: from pps.filterd (m0246630.ppops.net [127.0.0.1]) by mx0b-00069f02.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 2ANIO0g3016188 for <libc-alpha@sourceware.org>; Wed, 23 Nov 2022 21:08:12 GMT Received: from iadpaimrmta01.imrmtpd1.prodappiadaev1.oraclevcn.com (iadpaimrmta01.appoci.oracle.com [130.35.100.223]) by mx0b-00069f02.pphosted.com (PPS) with ESMTPS id 3m1nd890ym-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK) for <libc-alpha@sourceware.org>; Wed, 23 Nov 2022 21:08:12 +0000 Received: from pps.filterd (iadpaimrmta01.imrmtpd1.prodappiadaev1.oraclevcn.com [127.0.0.1]) by iadpaimrmta01.imrmtpd1.prodappiadaev1.oraclevcn.com (8.17.1.5/8.17.1.5) with ESMTP id 2ANJ6RfP010699 for <libc-alpha@sourceware.org>; Wed, 23 Nov 2022 21:08:11 GMT Received: from nam10-dm6-obe.outbound.protection.outlook.com (mail-dm6nam10lp2104.outbound.protection.outlook.com [104.47.58.104]) by iadpaimrmta01.imrmtpd1.prodappiadaev1.oraclevcn.com (PPS) with ESMTPS id 3kxnkdnxm5-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK) for <libc-alpha@sourceware.org>; Wed, 23 Nov 2022 21:08:11 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=O5PnHK8ijC62aeE8rdExAMYGXUHqtkwMS/5oP3ybkL/hfLiXp4iTcwG9g+69WJlZ1d1gucUHwnYr78G5a5jYjbxbzl8gEp9MIw4TFUK0/TjJTsNsnwEIyyt0sy71yYTRP/wyXLMC8J/Wbw8i/V/sDc94duBRKT6C/IrLUN48MiDwWC1gcGlwuWFa4GBxKOIPnsX0bphnszw3q/fM/zV5pZtfAZmtH80PWB2ukE5tuTVpobbKLf4PmMfRdeKeC0Qm/xsWlTdC+tMwoiUdC5xWjSyL2NzgORDXy+QfsGKksfZpXLpbc7WfhHJjwabEPk4ZskT6HBScdTFEXmAEVNThqA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=E3682w3fik55iGP3olNMtcepGGEPY7mpR/aJEzPWxBg=; b=bfvgNNIwwesSG774C6gQtc1vgsdqnuqPoQd7fpkI6GMBuR4W1k6o+lV0YT08dCAAFdSEciAFQ/Ko271ljgtnThhFiHH1H1ivpPfr5d6vfNDhLnrCnUZH0VBbaVoDIRcMkRcPQ1Wt+pFjDbuIBleIPgsB3J/ILtvGj/osLqgOP9Js0YFtPx82kSmVELRVAQIBt0+l4kh7uU3Oo3xfy1Vmrph+mewXbocFTioovPboPr9I5zfxKInRECHFThqybF6eswJ/xx5Y/uqQwDqVqYgIs8nJo2dB2GF8BvzKvd0ggCai1OnJzVCL5F7ZC/l6z6cfotDvd9AOyD2fCfK7m+x2Hg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none Received: from BYAPR10MB3208.namprd10.prod.outlook.com (2603:10b6:a03:159::10) by MW4PR10MB5812.namprd10.prod.outlook.com (2603:10b6:303:18e::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5857.17; Wed, 23 Nov 2022 21:08:09 +0000 Received: from BYAPR10MB3208.namprd10.prod.outlook.com ([fe80::9aae:27cb:b098:f6f6]) by BYAPR10MB3208.namprd10.prod.outlook.com ([fe80::9aae:27cb:b098:f6f6%4]) with mapi id 15.20.5834.015; Wed, 23 Nov 2022 21:08:09 +0000 Message-ID: <c4f64515-fec4-24dc-47a2-76b091718530@oracle.com> Date: Wed, 23 Nov 2022 15:08:05 -0600 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.5.0 Content-Language: en-US To: GNU C Library <libc-alpha@sourceware.org> Subject: [PATCH v2] Resolve-flockfile-funlockfile-differences X-ClientProxiedBy: SJ0PR03CA0087.namprd03.prod.outlook.com (2603:10b6:a03:331::32) To BYAPR10MB3208.namprd10.prod.outlook.com (2603:10b6:a03:159::10) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BYAPR10MB3208:EE_|MW4PR10MB5812:EE_ X-MS-Office365-Filtering-Correlation-Id: 6758eb7c-c7e5-4436-c081-08dacd96d2c3 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: k4GsiPpTIEfiGt5vVw144Y0eyaXc1qspNUBxlP4OYim0j1/mcv6lwevAkggg1EBT+KmCL7Z5NO6NQDC1pC7VbBnxSmfzaRMnDhtnNbsRxigik49CA32xyCXGcFchmA3AusbSvEnAFpaaeuRhe9EosOHOiaB9UyUYSFpKaWAFV+wvZXsoKtWQqzuX5apxWl/BguJRcos0IaUwFiyB0ftU8oBEjb+Y+6Zf/iHVd+cwXNVLMTf3DfD/mqGXTP+3U9g/W8NR1dHcQxJCphLMSj4xh+/Nx7EeZu8Ue7znmA669faQu5XHpONOmen/4LDV3Q3ibhqAbBC1FeyQxKq/j3ulMI+bWAENyGO/nFSW0BRaFh9dyytT+b2+xYY3SEuKSf7W2vDarMVWz67rmWQlCUQUczhBu9/z8C9TNELmCd7/c1XU9ghl4zyYYvHHhV6ePnNagG/vwknqeqpGpXV7OW38RQ== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BYAPR10MB3208.namprd10.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230022)(39860400002)(136003)(366004)(376002)(396003)(346002)(451199015)(6916009)(2616005)(186003)(316002)(31686004)(6512007)(26005)(6506007)(33964004)(6666004)(66946007)(564344004)(8676002)(66476007)(66556008)(38100700002)(83380400001)(36756003)(86362001)(8936002)(2906002)(6486002)(478600001)(31696002)(235185007)(41300700001)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?X6mggXYOIjoHdCh41ZGi8BLlg7LJ?= =?utf-8?q?kBUDL4uvTmviNkdBqPHlrpnSnuRE+0FzZDKIMAoq/A52gBZYKdsbtsEFIbxixOnOB?= =?utf-8?q?IJoMxV3V45aaQffxg9TnXQNkRfnbO+Zz3lM7Wc7D9mYBBSr4AqEssov769d1RBSHj?= =?utf-8?q?KxAPP/6Vveta0E466SL8FeoBKYs2EOPK/g8pOIaeovsuegrAhKvQlh23pqtDSb5aF?= =?utf-8?q?p3GsqU6Zx77mNjHYR1f268PWIeS/Aywmk6zTMoTz3qE0cm3RGZZ3DWSqI6+AoxO5t?= =?utf-8?q?G0EvsomKU26+9t8WWdYuhc41XjpSZX75sOkZCLj/OPlFv3aZ82to0FRKtDlSL86jO?= =?utf-8?q?KXMKiQI5CSwHkVsAGkLcWTwHJNnAzN106g9rr5YdMbT2RzacSo/Hvchlk1d8bEeOF?= =?utf-8?q?1oiIrkDLHOIq8QwQ6Fxx84ClyHrWMKgnoqmurE2FGZvtHPUOe1VAokF90Be+S563r?= =?utf-8?q?oCzP32rNR2YEAPoLWFUPCc2DaMjspTubZwB58aDi5l6jgyQAAs2msjrj2uY9BJBwp?= =?utf-8?q?wBMZCBYFKodpb0/Lik/6uivSrixC9Ke4tp7XA5xkr4d9ZN742pXW4fOAc/3n4r5De?= =?utf-8?q?2EqxxiVHPmxqfd20oc2J5ktEZKC6XArFtt8YryXZfcfSa7JwVl8zEg+Ar47Q883Zr?= =?utf-8?q?bQB59CvS8UJYD74gzyI7K+solxULBzhf6PKq+MfeMg/w/5qi4eYA+fe+vQvY22vYV?= =?utf-8?q?zQE1k4pNuRpuj9M444z832vyqXLJmnuRdRm6huEFcg52yh0Qpj9Taw1acsDgtsE9S?= =?utf-8?q?ase9zoQwKjXLqL/y66ql/hOMsu6YZCdWLOdkRUqMstdLuvgRjv2aHwZz8o3y93g2x?= =?utf-8?q?APHEgZfWjba3+2lLGrdLJNFK1xm6GOJvynk+GbSFQlbLXhsbyWB6B1eRiTVt9ZSch?= =?utf-8?q?Fy8rq/W8N5tnYXrG2QOCvF7pxRm+5nYprDhZOvffrVzvbomQxQPZcXlQ2RBIh8/0g?= =?utf-8?q?2C9SRVJ56mMkoIdKUXy19u0XDqPHJ1VjpjhtVSi6VkB8sFFPckZQWWEP6Kx7hUhMl?= =?utf-8?q?r/thxbXgFGA26yf1yz7H0NKsrW4ERnbP409JrMhL5qcM1Q2ZKKDqL5XwsyhQYyCVQ?= =?utf-8?q?gyORpKOHi+DQeoNA+Dh4v4WUSJ2fLplLGgN5XEKS38K57nGbN817aAXIcD+Ga2xcB?= =?utf-8?q?2w4YfJ2GeiQqKlnwe0LgvKRlru/kjWWNB9uHf/nD50iLUBAh72LLALjNrifMlpMPs?= =?utf-8?q?fX5HbGSz06QXvND0taHVWm9Bu2RqM5TZ+rNRY3/bwHBIsTr2Q2CBE1FQ8EhBQBdGT?= =?utf-8?q?98ofhh8ogOIAMPb/3DamaugtMuQ8VuhDmW2C105PT6znPAhg1G6N3KBpQaRpuM7sx?= =?utf-8?q?/V6UnEJUH9pFrR6mw4haam8W+7OzcWbKZzzXNtSzwK8ygj0OE1SPF6+IW3dy3eNH+?= =?utf-8?q?2wPr8/BgpyysW1oH/CDwilq606Iy6ZJJSNcHjGXIq6ZtSr6LDAKi3lSZA3LcTt9fK?= =?utf-8?q?LAM2lJHlW5LlQWnEriev1tdijDy8cPJA2e74q7awg2bpMxpVOhLYTdKVjmztS7/xm?= =?utf-8?q?tSNpgheAzvc4oJ+kwDR3PhYPvs9Kk5UwDQ=3D=3D?= X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: HMRuT6Jp2mjXtI/U1xQG/xR0SPeIDu4TUiUnieRNYfDBkzSOpkOZVUlP4OdAdNkinUFGsNeQEVaFy89SCVpYLWwC9XaoyM+pWp7N6ZWQOQeEDNkXZ8xpexFDcxjseYXY/pLo2cuEddZmCnyCX0pApQDXU+k3+sOY989uOar8s0aj4U8R7KAOz+Q7N3S+YwIWBqHE+6KwumqEJskM8asRi4hDuJOFgxfhvuOmBJUgaBfgFXqzICbiA4hxSDvBkqhBf2TVOYFp4J0L4Z5kEKqRTrDugsvfcw4J+4ykw6j9m/DyMPOURp0GvEsd8GQmPLoTfdFTca6PTT+fjTxMXhmLtWaVT26vzi/wCR5xt08ixQFlSpMSpv+9s17mcVn/9OY0wGnPNJiYl8WKKOi6+KtJxIaOHLYDB/N1EsMVVquWyN75YECS9bEdhRewwWQhSYS6jS37fGY8pcCoXYSwqiCJCr9eHFRITpbGKLA0TYXRNCKCLw5j8wcrylUotDkZpFJggJwjcfYiLfpjwOS5agm6nHSlJGPCVJLEFndByfyjupFH9WwH7pTH06F95qLtp+ORStqzYAH4ZJ4Dtc58bMGkgyNh0azMqmzs9PpNdP+EuLa+PK093+0a9JZATFv5sHyFlwzk+P8zk2qmzilh9TQNm3RG4IswcWUDre22mjNaFvLMQ+XSvlwyI0LBxOULi4PqGl/LgqECVaJHUTR/zyNk+tLLrR3S8oxdEP/dx5uhRbW/wRzg0CTubjYrJxHAEmfkQiKMGghMs0HqkrtOe367Qw== X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6758eb7c-c7e5-4436-c081-08dacd96d2c3 X-MS-Exchange-CrossTenant-AuthSource: BYAPR10MB3208.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Nov 2022 21:08:09.2585 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: aHRPIicndHQmXqH6IUk8lwlKmulZ5SJnSh1hsETDBEjPvlIlxVKTfBfvdElRplDAMNJq923entWjRy1JefdIXPiMMYmU5MaGztru7/opa9M= X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR10MB5812 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.219,Aquarius:18.0.895,Hydra:6.0.545,FMLib:17.11.122.1 definitions=2022-11-23_11,2022-11-23_01,2022-06-22_01 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 bulkscore=0 phishscore=0 adultscore=0 suspectscore=0 mlxscore=0 mlxlogscore=999 malwarescore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2210170000 definitions=main-2211230156 X-Proofpoint-GUID: vOYf0ry8W6s4iiC6bPHCse0vhiGKmR_Y X-Proofpoint-ORIG-GUID: vOYf0ry8W6s4iiC6bPHCse0vhiGKmR_Y X-Spam-Status: No, score=-12.5 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H2, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org Content-Type: text/plain; charset=UTF-8; name="0001-Resolve-flockfile-funlockfile-differences.patch" Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="0001-Resolve-flockfile-funlockfile-differences.patch" X-Content-Filtered-By: Mailman/MimeDel 2.1.29 X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list <libc-alpha.sourceware.org> List-Unsubscribe: <https://sourceware.org/mailman/options/libc-alpha>, <mailto:libc-alpha-request@sourceware.org?subject=unsubscribe> List-Archive: <https://sourceware.org/pipermail/libc-alpha/> List-Post: <mailto:libc-alpha@sourceware.org> List-Help: <mailto:libc-alpha-request@sourceware.org?subject=help> List-Subscribe: <https://sourceware.org/mailman/listinfo/libc-alpha>, <mailto:libc-alpha-request@sourceware.org?subject=subscribe> From: Patrick McGehearty via Libc-alpha <libc-alpha@sourceware.org> Reply-To: Patrick McGehearty <patrick.mcgehearty@oracle.com> Errors-To: libc-alpha-bounces+patchwork=sourceware.org@sourceware.org Sender: "Libc-alpha" <libc-alpha-bounces+patchwork=sourceware.org@sourceware.org> |
Series |
[v2] Resolve-flockfile-funlockfile-differences
|
|
Checks
Context | Check | Description |
---|---|---|
dj/TryBot-apply_patch | success | Patch applied to master at the time it was sent |
dj/TryBot-32bit | success | Build for i686 |
Commit Message
Patrick McGehearty
Nov. 23, 2022, 9:08 p.m. UTC
From a0291671db0d92a8762de3a45fd322e471a19a24 Mon Sep 17 00:00:00 2001
From: Patrick McGehearty <patrick.mcgehearty@oracle.com>
Date: Wed, 23 Nov 2022 21:02:02 +0000
Subject: [PATCH v2] Resolve flockfile/funlockfile differences
- - - - - -
Only difference from v1 is to correct indenting/tabs
for the changes to match other .c files in stdio-common.
- - - - - -
This patch resolves differences between flockfile/funlockfile and
LOCK_STREAM/UNLOCKSTREAM to avoid failure to unlock a stream mutex in
a multi-threaded application context which allows entering using
LOCK_STREAM but leaving using funlockfile or vice-versa.
This issue was detected during stress tests of a large proprietary
application. The cause and solution was identified by Gerd Rausch.
The issue occurs because _IO_funlockfile has different definitions in
different contexts:
Comparing the inline version in libio/libio.h
# define _IO_flockfile(_fp) \
if (((_fp)->_flags & _IO_USER_LOCK) == 0) _IO_flockfile (_fp)
And the non-inline version in stdio-common/flockfile.c
__flockfile (FILE *stream)
{
_IO_lock_lock (*stream->_lock);
}
Note the lack of the _IO_USER_LOCK in the __flockfile version. This
difference means it is possible to bypass the lock in some cases and
not release the lock in other cases. Either way, it causes trouble.
The proposed fix is to simple add the _IO_USER_LOCK guard to the
non-line versions of flockfile and funlockfile.
Modified files:
stdio-common/flockfile.c
stdio-common/funlockfile.c
---
stdio-common/flockfile.c | 3 ++-
stdio-common/funlockfile.c | 3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/stdio-common/flockfile.c b/stdio-common/flockfile.c index 49f72c6..7c82847 100644 --- a/stdio-common/flockfile.c +++ b/stdio-common/flockfile.c @@ -22,7 +22,8 @@ void __flockfile (FILE *stream) { - _IO_lock_lock (*stream->_lock); + if ((stream->_flags & _IO_USER_LOCK) == 0) + _IO_lock_lock (*stream->_lock); } weak_alias (__flockfile, flockfile); weak_alias (__flockfile, _IO_flockfile) diff --git a/stdio-common/funlockfile.c b/stdio-common/funlockfile.c index bf44c99..b77b1b2 100644 --- a/stdio-common/funlockfile.c +++ b/stdio-common/funlockfile.c @@ -23,7 +23,8 @@ void __funlockfile (FILE *stream) { - _IO_lock_unlock (*stream->_lock); + if ((stream->_flags & _IO_USER_LOCK) == 0) + _IO_lock_unlock (*stream->_lock); } weak_alias (__funlockfile, _IO_funlockfile) weak_alias (__funlockfile, funlockfile);