From patchwork Fri May 15 12:42:51 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Umang Jain X-Patchwork-Id: 3796 Return-Path: Received: from o1.f.az.sendgrid.net (o1.f.az.sendgrid.net [208.117.55.132]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id ED04260DF9 for ; Fri, 15 May 2020 14:42:53 +0200 (CEST) Authentication-Results: lancelot.ideasonboard.com; dkim=pass (1024-bit key; unprotected) header.d=uajain.com header.i=@uajain.com header.b="pUj1aQJi"; dkim-atps=neutral DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=uajain.com; h=from:subject:mime-version:to:cc:content-transfer-encoding: content-type; s=s1; bh=dTB+Mv4hA+Y+grxxDkDvIYM35BnAz1bOULSpDKQ+YEQ=; b=pUj1aQJimaFQeIdg3zmIl7n9CGsSpX1go1z7OpBaxXVmtAFVEnduLnAyPt7KbGzZlguo +1GzEHQyQ4GV4GFtUZUR6VEXyqrejx1LTLFSl+XuOPFK7n1bmarNohkTGmCIVg6BaMFHpQ 1brWzYw+bAZQcq4ALxkiDXnvFSZ4C0PIE= Received: by filterdrecv-p3iad2-8ddf98858-rpdh9 with SMTP id filterdrecv-p3iad2-8ddf98858-rpdh9-19-5EBE8E4B-4F 2020-05-15 12:42:51.825526115 +0000 UTC m=+4361717.575965514 Received: from mail.uajain.com (unknown) by ismtpd0001p1hnd1.sendgrid.net (SG) with ESMTP id NRVZ-Nk4RHSklyFmA2aWEg for ; Fri, 15 May 2020 12:42:51.332 +0000 (UTC) From: Umang Jain Date: Fri, 15 May 2020 12:42:51 +0000 (UTC) Message-Id: <20200515124245.18040-1-email@uajain.com> Mime-Version: 1.0 X-SG-EID: 1Q40EQ7YGir8a9gjSIAdTjhngY657NMk9ckeo4dbHZDiOpywc/L3L9rFqlwE4KPcHIuSzkf5/M5SksOT1pntG7LWoF/R7HMF+Bx9MtSCt5QjJrZSMznjkiqWjDTHUIpymejBn7VCzJvISqweAvfejPUhLjIGy3jjUytep5CRz+16PaVQDfTKjWKlJO+QvIEdagqpxSPl/4ToQEK+u2mXW49r/+PaFJDRQ9xDSDrc64U5pT/WbERDq9Nfrww6CTJQ/jG3+xfu2c1HJc2pcYQFvw== To: libcamera-devel Subject: [libcamera-devel] [PATCH v2 0/4] Coverity scan fixes X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 15 May 2020 12:42:54 -0000 Fixes of issues pointed out by coverity scans. Some of them were reviewed earlier (attached review tags for them) and some have been reworked to address the review. Umang Jain (4): test: ipc: unixsocket: Close open fds on error paths test: log: log_process: Close open fds on error paths test: log: log_api: Close open fds on error paths libcamera: camera: Return -EINVAL if any stream is null while configure() src/libcamera/camera.cpp | 5 ++++- test/ipc/unixsocket.cpp | 2 ++ test/log/log_api.cpp | 2 ++ test/log/log_process.cpp | 1 + 4 files changed, 9 insertions(+), 1 deletion(-)