From patchwork Thu Oct 7 10:04:45 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: GitHub issues - opened X-Patchwork-Id: 12541505 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 305A9C433EF for ; Thu, 7 Oct 2021 10:05:41 +0000 (UTC) Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id A3B44610E6 for ; Thu, 7 Oct 2021 10:05:39 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org A3B44610E6 Authentication-Results: mail.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=alsa-project.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=alsa-project.org Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id AF6D815E0; Thu, 7 Oct 2021 12:04:47 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz AF6D815E0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1633601137; bh=sVIdineeAMYdRUdlcwDeYTPTOFXAg6VtHflxLLASdp4=; h=From:To:In-Reply-To:References:Subject:Date:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=lg+FSYktgJjyPajqSf3TknzBnTCshtNnpL7u02Cwd/d08S+SS+bQsq2Hb2//38Ix5 kI6+2t3uZ7yfKX2xZoge+bABXErh18l5X8vdNEeqtCXzljmS3k0CKyctB3M1DHx8kd Vl0cDsjx1HyegGsO0iSs2QsxxJ0yBJ6KmMAu6I40= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 45869F80259; Thu, 7 Oct 2021 12:04:47 +0200 (CEST) Received: by alsa1.perex.cz (Postfix, from userid 50401) id D78D6F8027D; Thu, 7 Oct 2021 12:04:45 +0200 (CEST) Received: from webhooks-bot.alsa-project.org (gate.perex.cz [77.48.224.242]) by alsa1.perex.cz (Postfix) with ESMTP id 51B1BF80130 for ; Thu, 7 Oct 2021 12:04:39 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 51B1BF80130 MIME-Version: 1.0 From: GitHub issues - opened To: alsa-devel@alsa-project.org In-Reply-To: <1633601079600691080-webhooks-bot@alsa-project.org> References: <1633601079600691080-webhooks-bot@alsa-project.org> Subject: Double snd_dlobj_unlock() Message-Id: <20211007100445.D78D6F8027D@alsa1.perex.cz> Date: Thu, 7 Oct 2021 12:04:45 +0200 (CEST) X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" alsa-project/alsa-lib issue #181 was opened from johado: In dlmisc.c:snd_dlobj_cache_get0() there is a call to snd_dlobj_unlock() in the __err: path which should not be there. All lock/unlock are done by the callers to this function. Issue URL : https://github.com/alsa-project/alsa-lib/issues/181 Repository URL: https://github.com/alsa-project/alsa-lib --- a/src/dlmisc.c +++ b/src/dlmisc.c @@ -359,7 +359,6 @@ snd_dlobj_cache_get0(const char *lib, const char *name, free(c); __err: snd_dlclose(dlobj); - snd_dlobj_unlock(); return NULL; } c->dlobj = dlobj;