Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

crash when setting a wrong listenaddr #6384

Closed
monetdb-team opened this issue Nov 30, 2020 · 0 comments
Closed

crash when setting a wrong listenaddr #6384

monetdb-team opened this issue Nov 30, 2020 · 0 comments

Comments

@monetdb-team
Copy link

@monetdb-team monetdb-team commented Nov 30, 2020

Date: 2017-08-04 09:57:47 +0200
From: Zhouyang <>
To: Merovingian devs <>
Version: 11.27.5 (Jul2017-SP1)

Last updated: 2017-10-26 14:01:28 +0200

Comment 25547

Date: 2017-08-04 09:57:47 +0200
From: Zhouyang <>

User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_3) AppleWebKit/602.4.8 (KHTML, like Gecko) Version/10.0.3 Safari/602.4.8
Build Identifier:

After setting a wrong listenaddr, monetdbd will crash when starting.

Reproducible: Always

Steps to Reproduce:

  1. monetdbd create /a/new/dbfarm/path
  2. monetdbd set listenaddr=wronghost /a/new/dbfarm/path
  3. monetdbd start -n /a/new/dbfarm/path

Actual Results:

Aborted

Expected Results:

Exit and output proper log messages.

The reason is that there is no error handling code after library function "gethostbyname".
Attached please find the patch against version MonetDB-11.27.5.

Comment 25548

Date: 2017-08-04 10:00:25 +0200
From: Zhouyang <>

Created attachment 565
fix the crash by adding error handling code to gethostbyname

Attached file: monetdb-11.27.5-fix-crash.patch (text/plain, 575 bytes)
Description: fix the crash by adding error handling code to gethostbyname

Comment 25549

Date: 2017-08-04 10:41:59 +0200
From: MonetDB Mercurial Repository <>

Changeset fedb63ea4b33 made by Sjoerd Mullender sjoerd@acm.org in the MonetDB repo, refers to this bug.

For complete details, see https//devmonetdborg/hg/MonetDB?cmd=changeset;node=fedb63ea4b33

Changeset description:

Add error handling.
Fixes bug #6384.

Comment 25550

Date: 2017-08-04 10:42:22 +0200
From: @sjoerdmullender

Thanks for the patch. I added cleanup as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant