From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id kAoiKogXUGMHcA4AWB0awg (envelope-from ) for ; Wed, 19 Oct 2022 11:28:08 -0400 Received: by simark.ca (Postfix, from userid 112) id A91811E112; Wed, 19 Oct 2022 11:28:08 -0400 (EDT) Authentication-Results: simark.ca; dkim=pass (1024-bit key; secure) header.d=sourceware.org header.i=@sourceware.org header.a=rsa-sha256 header.s=default header.b=JKp5akbt; dkim-atps=neutral X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,RDNS_DYNAMIC,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 Received: from sourceware.org (ip-8-43-85-97.sourceware.org [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id 5E0011E0D3 for ; Wed, 19 Oct 2022 11:28:08 -0400 (EDT) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 0CD593857363 for ; Wed, 19 Oct 2022 15:28:08 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 0CD593857363 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1666193288; bh=9UtDGilkQltRfWFRUUzhjVVD4XRvpTIpMci0RF7Sf+w=; h=To:Subject:Date:In-Reply-To:References:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To: From; b=JKp5akbt9cp4f/w4ut2JahdIkkrkftRfOKIYORWRoQ0q9YPDkbKqH/epC1IIpFuje 0WBonyJHOM9lx4g8Ff612ak2QjgL10YIt+hfmQPOuzS5gHOBtpbLULE7HXCyDWsaM8 P01p3PFFFtuYPsEUAkjSA2n+6I0P5T+InNOIXQv8= Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by sourceware.org (Postfix) with ESMTPS id 52E733850862 for ; Wed, 19 Oct 2022 15:25:07 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 52E733850862 Received: from mail-qt1-f199.google.com (mail-qt1-f199.google.com [209.85.160.199]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_128_GCM_SHA256) id us-mta-478-6rpgTgsoMDqLE1VfX43Gzw-1; Wed, 19 Oct 2022 11:25:03 -0400 X-MC-Unique: 6rpgTgsoMDqLE1VfX43Gzw-1 Received: by mail-qt1-f199.google.com with SMTP id d1-20020ac80601000000b00388b0fc84beso12996632qth.3 for ; Wed, 19 Oct 2022 08:25:03 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=9UtDGilkQltRfWFRUUzhjVVD4XRvpTIpMci0RF7Sf+w=; b=nCNM9tTnN3HtrhJjxtyehhX6BJIhEValyVPHdeZv342bJ4CkWcekUK9H6zrPyKTZjd dCQuo+Fqaz67e8F3C7v8x3rQ9OwWgKesqulwE3rAqpPQwbkTJL/P+NSyAkrOaYj6Ps8X TzAxw6MnjzcKXuOsuy+GcWP8LIT37LtU6HiysYoS0AxaI4hq+3nFqjrL8yJFOlK2zLn/ yiS8d4JnN265W0ddvzx5t9l7OIuZpd62JJfetVJOvKHyt8bKnwRLbsCJF3AU+HJ/ipaq kAf7I8CgLHuwkAmJEjxFoeGuij9lfyVVodk+lFcswpKS4H84lieEpAbL7HbA4CdG23y6 79Vw== X-Gm-Message-State: ACrzQf3UWV4KWfU7mBrxp4QhcOnK//BljbEsr1eM3pFNHcmaU9SjqoKU /95vVAaVz45elsQYo/B6Gc+oKIGfIoSkJmFouHNyYe8vcCeQbVd/wKdxi8s6DSK1iSR3obaw/QJ KnxRcsbU57p3GBgEMyepNOj28r5fw+9KlnEJ4r+a4K3wDYQ62A9GtIhnYSseBXMqb0aG7DtiMyQ == X-Received: by 2002:a05:622a:206:b0:39c:f2cf:f9fa with SMTP id b6-20020a05622a020600b0039cf2cff9famr7031442qtx.669.1666193102783; Wed, 19 Oct 2022 08:25:02 -0700 (PDT) X-Google-Smtp-Source: AMsMyM7T+z+q/RQdrchsWblUiJJLxIxbAerrhbo9EvI5Cny03pOfa9R+YklJW2ZC7vRODKrk0Y8tVg== X-Received: by 2002:a05:622a:206:b0:39c:f2cf:f9fa with SMTP id b6-20020a05622a020600b0039cf2cff9famr7031419qtx.669.1666193102503; Wed, 19 Oct 2022 08:25:02 -0700 (PDT) Received: from localhost ([31.111.84.238]) by smtp.gmail.com with ESMTPSA id t12-20020ac8588c000000b0039cd4d87aacsm4314013qta.15.2022.10.19.08.25.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 19 Oct 2022 08:25:02 -0700 (PDT) To: gdb-patches@sourceware.org Subject: [PATCH 04/10] sim/ppc: don't pass uninitialized value to semctl for GETVAL calls Date: Wed, 19 Oct 2022 16:24:43 +0100 Message-Id: <5155a6bb36800aa1a8bdb214f59f85f9d92b6989.1666192979.git.aburgess@redhat.com> X-Mailer: git-send-email 2.25.4 In-Reply-To: References: MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="US-ASCII"; x-default=true X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Andrew Burgess via Gdb-patches Reply-To: Andrew Burgess Errors-To: gdb-patches-bounces+public-inbox=simark.ca@sourceware.org Sender: "Gdb-patches" When calling semctl with an GETVAL action, the 'union semun' argument is unused. Rather than passing an uninitialized variable (as we currently do), just pass the integer 0. This silences some compiler warnings, and is just as correct (given the argument is ignored). To be honest, it might be the case that the argument is not needed at all, however, I'm a little nervous to make this change as the amount of testing I can do of this code is limited. Every example I found online for a semctl/GETVAL call did pass the final argument, so that's what I've continued to do. --- sim/ppc/hw_sem.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/sim/ppc/hw_sem.c b/sim/ppc/hw_sem.c index 937e2ad6f81..78e3dda9ae6 100644 --- a/sim/ppc/hw_sem.c +++ b/sim/ppc/hw_sem.c @@ -148,7 +148,7 @@ hw_sem_init_data(device *me) error("hw_sem_init_data() semget failed\n"); } - sem->count = semctl( sem->id, 0, GETVAL, help ); + sem->count = semctl( sem->id, 0, GETVAL, 0 ); if (sem->count == -1) error("hw_sem_init_data() semctl -- get value failed\n"); DTRACE(sem, ("semaphore OS value (%d)\n", sem->count) ); @@ -241,7 +241,7 @@ hw_sem_io_read_buffer(device *me, } /* assume target is big endian */ - u32 = H2T_4(semctl( sem->id, 0, GETVAL, help )); + u32 = H2T_4(semctl( sem->id, 0, GETVAL, 0 )); DTRACE(sem, ("semaphore OS value (%d)\n", u32) ); if (u32 == 0xffffffff) { -- 2.25.4