From patchwork Thu Apr 24 12:06:16 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bartosz Golaszewski X-Patchwork-Id: 884087 Received: from mail-wr1-f49.google.com (mail-wr1-f49.google.com [209.85.221.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 338471FC0F3 for ; Thu, 24 Apr 2025 12:06:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.49 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1745496391; cv=none; b=cX1kvZgeWQyS6qjfi84FwUF7lihgVfJgqMkYh0VeV18mNrULiyuGk7q+kAkx9ruR8J1pLFH4LbnVKiLnDAgIGQePXjfny2IR/p42JVVipXbL+A5EvL3g418S4nDjUJwXmyL05KTxn6vdTBkvE0aZjIEVOr7JL5j4FN5Hzb+WIZA= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1745496391; c=relaxed/simple; bh=E17apN4/FvY8+8WPROx8SQVb30orZsn+3FR7H40wutc=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=UMQlR1f5ISSLgzGNnYqWqKW12owV39R9CPQzmiz2MHZEgZIDXKYEo28HgEPpLOsEkyV43/K5TWcg/2y9+nl5PTHi+cC5S7b2O4JA0yQKMHhow5d0euGsONBPpQtuSSqme4W1IUcQ8ow+7ifdKDt6xvDnx5IPS+myHIpFJhP0SVA= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl; spf=none smtp.mailfrom=bgdev.pl; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b=zDzjvDTY; arc=none smtp.client-ip=209.85.221.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b="zDzjvDTY" Received: by mail-wr1-f49.google.com with SMTP id ffacd0b85a97d-39c0dfba946so662116f8f.3 for ; Thu, 24 Apr 2025 05:06:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20230601.gappssmtp.com; s=20230601; t=1745496388; x=1746101188; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=2JZ1tbc2jldm1ypA78cgb9SagnhmvfhXzqqpq8uOXtg=; b=zDzjvDTYZkIB+90c9eOR6EoaAZqmRedHFjvtexiM93najUKEUkuqsD08YVFluBk9e4 7DfFBfBY+02DI1tOgisgf3P3O7e/yJTRoh2YmE6jpBb5/pvxy0HbGDkFqkqHeU89pC+J mGLI/XnokWsZptPePSJdlf29e24kBkDXZGEeLmn8tcpJAM9LNbOW+dw5FVLPP1xt0KUH AXp2sTa1sK02enVxxPkmNsLvOiwD6jzOphvMdpbVJrqGgt/LkO7bt633qcop2QkJjPHw 0pUCrsuhdqvKxYwz16Y4W7O+cdWIFijLDehKiEIc56oVy8sc3x1d3OaMKubBtauqo66s IOqg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1745496388; x=1746101188; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=2JZ1tbc2jldm1ypA78cgb9SagnhmvfhXzqqpq8uOXtg=; b=jKGxAEhJItU693HejHK2ESdhnBT5zSfWSwqDgYKfKamq7IDPp8QC2jQV5gcikwiudk UX/2jR0We7k2tznBQwRw8LpSETPfXQCguYxF+iNBkOjFDEld7wfBOarxSf68JHWfdIAn UkhhG8wqOhlo+fCmNfZ1ci1B1BUy4dM5nyhBm9ZYaojHZBRDhMuJZyeb4fqMk1yZjUyL 0te8D3ALGJyXgRTbAsctKbBG8zzm6nsLiuvz8RqgdDG+2/Vs0lqtUJTVG52cLDW0M+bF buAgeEFFWMXmKp7fc6sFIdYRGIjp+8E72dBekcGSFLJIsyuKaCD87zd0WsZn5ZuqEA6D IhNA== X-Gm-Message-State: AOJu0Yw4ry+XRV1Gt32HBjFYGYXkrCM+cAzztHhWm3rbKBPFlLeUeH9y RpnIsbHGHGMx67/pEZcxj1DNHQN2NvWSclZC7taD3AJs2NRl8kOfhkkQoU/pZ30= X-Gm-Gg: ASbGnct29hsmiZ6Ik2YICSSpD0OFjNo9AxfhGxVzQakfh0S6iIPWetg/cUtxYMXlir4 bZcS4TTZ/tQAiEIMEzkv2Ozbl0mL4526J5Rq7R88wosSPpKBmujbGZs/oeSxPl4d3B02mxyjZOW ydtAHwS6oKsXlXKKlaolO7jXhbZr+fgGWWqUU5vmDO1lDHjU2eTm232WIyawYlysZvGnge3Auzr ZULD0g2cjQqG1D2XAmhkB3qjkoVLGVlmasNI+/aO9ve1pkLbit4tk5qWpVlwWXDuQULutTIPyYY 7ly81sd2hyFntcSe43IPlfccsfse1GgDqQ== X-Google-Smtp-Source: AGHT+IGsG4VUEswSS1V9LYbzvJK+dG3A5geKva6wNQIYeFolH/3qhfbZCVpBQ76Fy8TXkLKuokPKow== X-Received: by 2002:a05:6000:438a:b0:39e:e557:7d9 with SMTP id ffacd0b85a97d-3a06cf4b69emr1977975f8f.5.1745496388214; Thu, 24 Apr 2025 05:06:28 -0700 (PDT) Received: from [127.0.1.1] ([2a01:cb1d:dc:7e00:6266:7750:57ce:7cb4]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3a06d4c108esm1881793f8f.40.2025.04.24.05.06.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 24 Apr 2025 05:06:27 -0700 (PDT) From: Bartosz Golaszewski Date: Thu, 24 Apr 2025 14:06:16 +0200 Subject: [PATCH libgpiod 1/2] core: keep memory usage of struct line_config in check Precedence: bulk X-Mailing-List: linux-gpio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20250424-line_config_mem_use-v1-1-fa0abdcf0cdf@linaro.org> References: <20250424-line_config_mem_use-v1-0-fa0abdcf0cdf@linaro.org> In-Reply-To: <20250424-line_config_mem_use-v1-0-fa0abdcf0cdf@linaro.org> To: Kent Gibson , Linus Walleij Cc: linux-gpio@vger.kernel.org, Bartosz Golaszewski X-Mailer: b4 0.14.1 X-Developer-Signature: v=1; a=openpgp-sha256; l=2383; i=bartosz.golaszewski@linaro.org; h=from:subject:message-id; bh=OpiXCiS8GEaSSqvMr+cMwA6cA9s2uZH8AiFO6/NdaCA=; b=owEBbQKS/ZANAwAKARGnLqAUcddyAcsmYgBoCilCk2YJ2IxON8nBtlOaLTzvyKjQQ4CN/MHk/ Or/xEJvcNyJAjMEAAEKAB0WIQQWnetsC8PEYBPSx58Rpy6gFHHXcgUCaAopQgAKCRARpy6gFHHX cu0aD/oCVdq+nnfqm/V5K8/TK89LAEmIyTmZho+7XJ+izrnN3CNRDjTmglT2ufJTfrLPDQXV1l1 9LpSN/wJ0/SZxlcDOOxcMzFG08HYlbBw/UxZOpMgR9GkN8HfWrhG8od9Mi62tAOFLLiSdhri10k QUoEuXxgwmAOsHWjAea9UE9QtgbcUSMFqHyPUpKzN58FzoefgUvrt+gBBgSMJrFzMg/WaNymNV3 zEMIsCdL2gUjaD7eeU7gV0vb2QVqrz/ns9Ha8VQROI3RWYfR3ggB2ktZ46Dtk9ZEHKtU7JAcdbe wBkCE9DPiQ1GLbcpBZMKe93lB29NOQiL06eccBYe8zrn8XONPz51HvrsC47CPRGaiAjieYCCcVD tOnMvQUIX/kAprUkdHhylJr9NQZ2JMtgPlvU9dS5OzVFJKG9rNBuTxvxBsJNIG+rrTiicZoF3M9 O/Y1zfSiTyOUrB0o8AboKyST+dtDSSEuHqw3+PH8Oxn8tyebsqXkStBT2oPhx0n0smLxoqK56ma OQAQj0Y1iqnh2QwAt32H+PeXnbBl/onAizj9AQ/Tdm5L6jfswXBG23399U4IuXrqxUTAHa70oSO IwCJjgyuOvYZWwcFbrBVrOUz74rO9Mzj3lJFlp9kMdzByBg15n/HnysIVUBcnLs1v1t26oA8K4g qpLjTUmBIHE1LfA== X-Developer-Key: i=bartosz.golaszewski@linaro.org; a=openpgp; fpr=169DEB6C0BC3C46013D2C79F11A72EA01471D772 From: Bartosz Golaszewski We keep a single instance of a struct gpiod_line_settings object shared by all offsets using it. Everytime gpiod_line_config_add_line_settings() is called, we append it to the per-line-config linked list and reference it from every per-offset structure. However we don't keep track of the offsets referencing a settings object and don't remove it when there are no more offsets associated with it - we only do it when the user frees or resets the line-config object. Extend the settings_node structure: make it a doubly-linked list - allowing to easily remove individual nodes - and add a reference count. Keep track of how many offsets reference a line-settings object and when that reference count drops to 0, remove it. Fixes: b7ba732e6a93 ("treewide: libgpiod v2 implementation") Closes: https://github.com/brgl/libgpiod/issues/130 Signed-off-by: Bartosz Golaszewski --- lib/line-config.c | 19 ++++++++++++++++++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/lib/line-config.c b/lib/line-config.c index 9302c1b..24357f3 100644 --- a/lib/line-config.c +++ b/lib/line-config.c @@ -13,8 +13,10 @@ #define LINES_MAX (GPIO_V2_LINES_MAX) struct settings_node { + struct settings_node *prev; struct settings_node *next; struct gpiod_line_settings *settings; + unsigned int refcnt; }; struct per_line_config { @@ -93,7 +95,7 @@ GPIOD_API int gpiod_line_config_add_line_settings( size_t num_offsets, struct gpiod_line_settings *settings) { struct per_line_config *per_line; - struct settings_node *node; + struct settings_node *node, *old; size_t i; assert(config); @@ -121,14 +123,29 @@ GPIOD_API int gpiod_line_config_add_line_settings( return -1; } + node->refcnt = 0; node->next = config->sref_list; + if (config->sref_list) + config->sref_list->prev = node; + node->prev = NULL; config->sref_list = node; for (i = 0; i < num_offsets; i++) { per_line = find_config(config, offsets[i]); + node->refcnt++; per_line->offset = offsets[i]; + old = per_line->node; per_line->node = node; + + if (old && !(--old->refcnt)) { + if (old->prev) + old->prev->next = old->next; + if (old->next) + old->next->prev = old->prev; + free(old->settings); + free(old); + } } return 0; From patchwork Thu Apr 24 12:06:17 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bartosz Golaszewski X-Patchwork-Id: 885112 Received: from mail-wr1-f44.google.com (mail-wr1-f44.google.com [209.85.221.44]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C0696221FCC for ; Thu, 24 Apr 2025 12:06:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.44 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1745496392; cv=none; b=cbTzFBFrNTzSumwmwkFw21x5QLxSSmLljboygplBWf9+w37bMc397vN54sOmv5oQl4BOhzkU+4NLff1iq/OEPpUqZKOGrBDK3vh7lXrEwNbg4LI15i9kdbjgabm1LpiDDWz20srCySGEhs/KzsxqyDyl0vlE3T1Tqfh7BzRujp4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1745496392; c=relaxed/simple; bh=fZPyM+t+8g32uOJLovcOdzH0PaxIlPmC8Vhht+AXf+E=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=TnoN+dSA2onkytTapqRgIMBudgUXReJx5TnVMZUFdKxSVfI3cupjRbNk0QVm6YJH+VoRPs6Hc609i1Hdlwv6hEW8BU2rP20H3cJ+qLVt9V9Q2PyUgOTgwXJG/JtBRQe5GlrbXDXup9cXCVXM1Y+QN4pYlDx0LebY/LaOxGeddbQ= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl; spf=none smtp.mailfrom=bgdev.pl; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b=QTuLnxeL; arc=none smtp.client-ip=209.85.221.44 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=bgdev.pl Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bgdev-pl.20230601.gappssmtp.com header.i=@bgdev-pl.20230601.gappssmtp.com header.b="QTuLnxeL" Received: by mail-wr1-f44.google.com with SMTP id ffacd0b85a97d-3912fdddf8fso1428636f8f.1 for ; Thu, 24 Apr 2025 05:06:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20230601.gappssmtp.com; s=20230601; t=1745496389; x=1746101189; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:from:to:cc:subject:date:message-id :reply-to; bh=XmBfepC1ocfC4zLY8rWZfc6OY5eKgwPba73i3bJDtkU=; b=QTuLnxeLORl0wmOTvLphxiZCls6/RxuVdeuzEq37k/ffWpF40BLH6DYe2t7PjeuhWJ XM7rzG58yYy6aYI82UUSYZVNpAFEBtrhBjKTB9kEDtO0qPlXcB9YA1CH06EYrhhGF4I3 RHf0n80/vH59fJbTwxIt6yS7947trd8e8U3cYAKuAtUhEI8hawG0EyUiSdgV7k0XDrF3 NRKgLJHdpGDlR1NpvCUMcETItzIg4Zl0z+c4ggwJGBAn40BMB/dB+jj+yRT9K58o4jGZ ztfm+oGNS8SpEMSiQ0nJxQfLJZ/dAJbFs3doUQ22F5Kd5/MNDgwrNpx15TLb9VIZARyE MdHg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1745496389; x=1746101189; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :mime-version:subject:date:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=XmBfepC1ocfC4zLY8rWZfc6OY5eKgwPba73i3bJDtkU=; b=DgcTFb57TpPjqJ9zhDS/v/VdWIxtWhOkoCddRMANpCWbLmbo7Qbx9G6hVzSvvBaF/b KqLFQ8uPcCsbg5uxMyhmb24zP6c8KjBJ7C/Pj+0fCj843Vb/twYaUT+xSUhmIW1FKkGj dBh/b6nrZWi2cDnc8bPRKiMGsP0x2PeHliRRyIyjYYQeu8Vahr0L0CIe9OODEfb4IkgM jzoCvYf4PmoCquPw/2Aw34prbk7XTc94iHlNuAZ1JjKK5HCp7KWIU1VPRWLfJZ73SYAj KxW5LjECrCxuCSpLs6CJHxZr1MUCobqtb696dGWyYDWWFGOqZDNQWjEC32PQJ7gssqi5 20Qw== X-Gm-Message-State: AOJu0YxdW75Sp1wbkCXy53t6O+dQBlp29WJlXipcM9yvMVyNKMPZNMgR MuTXVLPLt925k4kAKn3fTd6Pmr2w5pSCp97bfrAvoz/b/GrONDxZFBHmrBNjBEM= X-Gm-Gg: ASbGncvvc76D1esUqc3RW+kj23Z3wzR3HxUBsmlKlu1tRL2+Idu4P1HWrOu6XfNP1Rf XM0LSm50pLMZ5u+FMhWk1xLMGYNOI3e0X07nXyXyfh/OekR1XZvo/6BE1JFGlzd1VjYpSZ99VhZ jNKISjMT9yZYeYuNnWDSUlb1ZVKSZ+PSaJceU+nEcFOyGwXzuXC1YVfFUhwDRvZ6j9+LYM7kLGO MHzip6cfAklFWdR8i3AeiW0remPwvuAmycSAiYgcLlXYCGlfO6i62LzUr0ul0GyLdFEWEsb9xO+ atzSsduCb4o9H1CUE1iDRoB06PcbtlQP4g== X-Google-Smtp-Source: AGHT+IFox1VUSgU/q9Vx6891P6KlOnQiiX9b7DUasucUXkg7XX/1HCJB9DC3kj5S7UsebUQ0eOubJg== X-Received: by 2002:a5d:648a:0:b0:39f:d0a:5b23 with SMTP id ffacd0b85a97d-3a06d674accmr2099066f8f.17.1745496388937; Thu, 24 Apr 2025 05:06:28 -0700 (PDT) Received: from [127.0.1.1] ([2a01:cb1d:dc:7e00:6266:7750:57ce:7cb4]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3a06d4c108esm1881793f8f.40.2025.04.24.05.06.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 24 Apr 2025 05:06:28 -0700 (PDT) From: Bartosz Golaszewski Date: Thu, 24 Apr 2025 14:06:17 +0200 Subject: [PATCH libgpiod 2/2] tests: add a test-case checking line-config memory usage Precedence: bulk X-Mailing-List: linux-gpio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Message-Id: <20250424-line_config_mem_use-v1-2-fa0abdcf0cdf@linaro.org> References: <20250424-line_config_mem_use-v1-0-fa0abdcf0cdf@linaro.org> In-Reply-To: <20250424-line_config_mem_use-v1-0-fa0abdcf0cdf@linaro.org> To: Kent Gibson , Linus Walleij Cc: linux-gpio@vger.kernel.org, Bartosz Golaszewski X-Mailer: b4 0.14.1 X-Developer-Signature: v=1; a=openpgp-sha256; l=2729; i=bartosz.golaszewski@linaro.org; h=from:subject:message-id; bh=WHEXTrJML2n86+yPEpXIRKRki8p0tCWD5RkbP3HBkhk=; b=owEBbQKS/ZANAwAKARGnLqAUcddyAcsmYgBoCilCsvj1Kg67WxXVUmnq03wYmDGvMvU+wbEgF okvsPzYGK6JAjMEAAEKAB0WIQQWnetsC8PEYBPSx58Rpy6gFHHXcgUCaAopQgAKCRARpy6gFHHX cnX/D/sF9iloOJeoRLwlxWWI5oy2uVCnX2iwHLmIoHV2K/4Atx75J0rNz02RRQz1JSL96gMbDOR HAzsXgoIIyjeDHQP3k4VQHdK5HWatyFuwsrfmNXygnZxvuaUfCjCC/Hac0VZVy2Uzt3mpCTl/Wo nQ+K36ZiZmpt8/Ne/bi/mSplIUti68JqETV8AsaPlXBX03E1KZPQXhYyypdyHVQIOgm3k1NJ48F Gm8w5rpKSraDHnipk5OxziPd7KG1DsXjtKFNtpMGvxFSCpfrCUnBoApziEwjkyIZfINoXoCVo87 quRstI79MLyIVE4hKMyQoGvttAZV+YzPjvwT6V6hA3gqsGx8nREs98i8yo4H3XPeFt5/dagSU0O 3aeSa+zSrVqn71gb+aduDRbi2PB/GIuplNSgVizDvlYryzae7GVoOANRcMSJggheIg304qebQvZ 93U0WiTnauPso37vqIg5k1E2YINVBim4czvAXhkcPOmgO4yqwvh3/5JXnzIH/BIxTkNgOiyQmYZ zO88ZhRiJnCqZlnWeFOLkjobFfxMLqpVEJtbg5/bTp0Sf4uviqmoQ8or1nGBEILp0zju+vaUMsp dRibJDo7p30wGwUM2lAquqC1HH4K8t9UbEonMmmo8ohKa4XwzFhUGKk7+rgsvnLWhZXhSdGTvra 247BN15BQ0WQavA== X-Developer-Key: i=bartosz.golaszewski@linaro.org; a=openpgp; fpr=169DEB6C0BC3C46013D2C79F11A72EA01471D772 From: Bartosz Golaszewski Add a test case making sure that repeatedly adding line settings to a line-config object doesn't continuously inflate its size until OOM. Signed-off-by: Bartosz Golaszewski --- configure.ac | 1 + tests/tests-line-config.c | 43 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 44 insertions(+) diff --git a/configure.ac b/configure.ac index af5d53d..416ae28 100644 --- a/configure.ac +++ b/configure.ac @@ -172,6 +172,7 @@ then AC_CHECK_FUNC([tsearch], [], [FUNC_NOT_FOUND_LIB([tsearch])]) AC_CHECK_FUNC([tdestroy], [], [FUNC_NOT_FOUND_LIB([tdestroy])]) AC_CHECK_FUNC([tdelete], [], [FUNC_NOT_FOUND_LIB([tdelete])]) + AC_CHECK_FUNC([setrlimit], [], [FUNC_NOT_FOUND_LIB([setrlimit])]) AC_CHECK_HEADERS([sys/utsname.h], [], [HEADER_NOT_FOUND_LIB([sys/utsname.h])]) AC_CHECK_HEADERS([sys/mount.h], [], [HEADER_NOT_FOUND_LIB([sys/mount.h])]) AC_CHECK_HEADERS([sys/prctl.h], [], [HEADER_NOT_FOUND_LIB([sys/prctl.h])]) diff --git a/tests/tests-line-config.c b/tests/tests-line-config.c index b61a445..b510e0f 100644 --- a/tests/tests-line-config.c +++ b/tests/tests-line-config.c @@ -7,6 +7,8 @@ #include #include #include +#include +#include #include "helpers.h" @@ -469,3 +471,44 @@ GPIOD_TEST_CASE(handle_duplicate_offsets) g_assert_cmpuint(retrieved[1], ==, 2); g_assert_cmpuint(retrieved[2], ==, 3); } + +GPIOD_TEST_CASE(dont_allow_line_config_to_balloon_out_of_control) +{ + static const guint offsets[] = { 0, 2 }; + + g_autoptr(struct_gpiod_line_config) config = NULL; + g_autoptr(struct_gpiod_line_settings) settings = NULL; + struct rlimit rlim; + guint counter; + gint ret; + + /* + * Severly limit the available virtual memory and make sure that + * adding settings for the same offsets repeatedly does not trigger + * the OOM killer. + */ + + if (g_test_subprocess()) { + config = gpiod_test_create_line_config_or_fail(); + settings = gpiod_test_create_line_settings_or_fail(); + + rlim.rlim_cur = rlim.rlim_max = sysconf(_SC_PAGESIZE) * 100; + ret = setrlimit(RLIMIT_AS, &rlim); + g_assert_cmpint(ret, ==, 0); + gpiod_test_return_if_failed(); + + for (counter = 1000000; counter; counter--) { + gpiod_line_settings_set_direction(settings, + counter % 2 ? + GPIOD_LINE_DIRECTION_OUTPUT : + GPIOD_LINE_DIRECTION_INPUT); + gpiod_test_line_config_add_line_settings_or_fail( + config, offsets, 2, settings); + } + + return; + } + + g_test_trap_subprocess(NULL, 0, G_TEST_SUBPROCESS_DEFAULT); + g_test_trap_assert_passed(); +}