Discussion:
Potential memleak in bearerbox
Paul Bagyenda
2018-10-18 09:38:45 UTC
Permalink
Content preview: Hi,  In bb_boxc.c, both smsbox_start() and wapbox_start()
initialise box_allow_ip/box_deny_ip. Which means if you have WAP and SMS
running, you leak a few bytes. Caught this using GCC LSAN. Quick fix wa [...]


Content analysis details: (0.0 points, 3.0 required)

pts rule name description
---- ---------------------- --------------------------------------------------
X-BeenThere: ***@kannel.org
X-Mailman-Version: 2.1.21
Precedence: list
List-Id: Development mailing list <devel.kannel.org>
List-Unsubscribe: <http://www.kannel.org/mailman/options/devel>,
<mailto:devel-***@kannel.org?subject=unsubscribe>
List-Archive: <http://www.kannel.org/pipermail/devel/>
List-Post: <mailto:***@kannel.org>
List-Help: <mailto:devel-***@kannel.org?subject=help>
List-Subscribe: <http://www.kannel.org/mailman/listinfo/devel>,
<mailto:devel-***@kannel.org?subject=subscribe>
Errors-To: devel-***@kannel.org
Sender: "devel" <devel-***@kannel.org>

Hi,

 In bb_boxc.c, both smsbox_start() and wapbox_start() initialise
box_allow_ip/box_deny_ip. Which means if you have WAP and SMS running,
you leak a few bytes. Caught this using GCC LSAN. Quick fix was/is to
only initialise them if they are NULL.


P.

Loading...