From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from m126.mail.126.com (m126.mail.126.com [220.181.12.36]) by sourceware.org (Postfix) with ESMTP id EEEE03858421 for ; Tue, 20 Dec 2022 13:08:50 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org EEEE03858421 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=126.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=126.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=126.com; s=s110527; h=From:Subject:Date:Message-Id:MIME-Version; bh=Z1yqt vbSCV+4vaW7H7AcIbu23Af9CFaVGPKpazd3JIs=; b=YQfKipwxPRS32cEd8i6SX Mo4cKrJBHXm2AH+ffjU1c/XZswvUwKmlj+26CGwLk51YhaJCBaUvUPFwDVCPm8si GFoTNsCyOJWNzyGHyVaRdQR8MZmw7W/BnghuekUfcpFamHcXy6wdqt8b4ogDd6yD kRgb9xj7g/9zQdc2444lWw= Received: from localhost.localdomain (unknown [111.32.120.106]) by zwqz-smtp-mta-g2-0 (Coremail) with SMTP id _____wDHnpJus6Fj1bMWAA--.27965S2; Tue, 20 Dec 2022 21:07:01 +0800 (CST) From: Xiaole He To: libabigail@sourceware.org Cc: Xiaole He , Xiaole He Subject: [PATCH] elf-reader: reclaim fd and mem before break Date: Tue, 20 Dec 2022 21:06:34 +0800 Message-Id: <20221220130634.9693-1-hexiaole1994@126.com> X-Mailer: git-send-email 2.27.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-CM-TRANSID:_____wDHnpJus6Fj1bMWAA--.27965S2 X-Coremail-Antispam: 1Uf129KBjvJXoW7CFykWw4fXFW8uF4DKr18uFg_yoW8Wr1Dpa y3uw1UKr4UJr1fCry3C3yUXF9Iqayjqa1Uur9I9343tr9xJr9I9rWfJrWfGFWjqFZ7W34Y qr43XrWxAa48Aw7anT9S1TB71UUUUUUqnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDUYxBIdaVFxhVjvjDU0xZFpf9x0pEs2-bUUUUU= X-Originating-IP: [111.32.120.106] X-CM-SenderInfo: 5kh0xt5rohimizu6ij2wof0z/1tbiThzdBmIxk21uXgAAsU X-Spam-Status: No, score=-10.7 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_ENVFROM_END_DIGIT,FREEMAIL_FROM,GIT_PATCH_0,SPF_HELO_NONE,SPF_PASS,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 List-Id: In 'src/abg-elf-reader.cc': /* src/abg-elf-reader.cc begin */ 1 void 2 locate_alt_ctf_debug_info() 3 { 4 ... 5 for (const auto& path : rdr.debug_info_root_paths()) 6 { 7 ... 8 int fd; 9 if ((fd = open(file_path.c_str(), O_RDONLY)) == -1) 10 continue; 11 12 ... 13 Elf *hdl; 14 if ((hdl = elf_begin(fd, ELF_C_READ, nullptr)) == nullptr) 15 ... 16 17 alt_ctf_section = 18 elf_helpers::find_section(hdl, ".ctf", SHT_PROGBITS); 19 break; 20 21 elf_end(hdl); 22 close(fd); 23 } 24 ... 25 } /* src/abg-elf-reader.cc end */ The file descriptor 'fd' and the memory that 'hdl' pointed to can have a chance where they were only created but nerver been destroyed when above code reach the line 19. Thus cause the leakage of file descriptor and memory. This leakage problem had already occured on our system, and the problem finally cause process can not open any more file and complaint 'Errno 24: Too many open files'. This patch fix above problem. * src/abg-elf-reader.cc (locate_alt_ctf_debug_info): reclaim fd and mem before break. Signed-off-by: Xiaole He --- src/abg-elf-reader.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/abg-elf-reader.cc b/src/abg-elf-reader.cc index c07f0655..979f0aae 100644 --- a/src/abg-elf-reader.cc +++ b/src/abg-elf-reader.cc @@ -453,10 +453,10 @@ struct reader::priv // unlikely .ctf was designed to be present in stripped file alt_ctf_section = elf_helpers::find_section(hdl, ".ctf", SHT_PROGBITS); - break; elf_end(hdl); close(fd); + break; } } -- 2.27.0