From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: location@lists.ipfire.org
Subject: Re: Problem with 'loc_as_get_name()'
Date: Fri, 23 Oct 2020 10:49:09 +0100
Message-ID: <3C3541B3-64F4-4077-B13B-FBE63040784C@ipfire.org>
In-Reply-To: <ff01ae4b-78e7-5d2f-e62c-44957eb8f5d8@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============2089235709950895497=="
List-Id: <location.lists.ipfire.org>

--===============2089235709950895497==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Good morning,

> On 22 Oct 2020, at 14:39, Gisle Vanem <gisle.vanem(a)gmail.com> wrote:
>=20
> I've changed the subject since it's now off-topic
> (solved that part).
>=20
>> And a CRT is what?
>=20
> A "C Run-time".
>=20
>=20
>> +       // Set country code
>>> +       loc_network_set_country_code(network4, "ZZ");
>=20
>> I assume you meant to set the country code for network5?
>=20
> Oh, yes. Silly me.
>=20
> But other item. I've come a long way integrating 'libloc'
> support in my program. Basically I do:
>=20
>  struct in6_addr addr;
>  struct loc_network *net;
>  struct loc_as      *as =3D NULL;
>  ...
>  loc_database_lookup (libloc_data.db, &addr, &net);
>  int         family  =3D loc_network_address_family (net);
>  const char *country =3D loc_network_get_country_code (net);
>  const char *AS_name =3D NULL;
>=20
>  uint32 AS_num =3D loc_network_get_asn (net);
>  is_anycast    =3D loc_network_has_flag (net, LOC_NETWORK_FLAG_ANYCAST);
>  is_anon_proxy =3D loc_network_has_flag (net, LOC_NETWORK_FLAG_ANONYMOUS_PR=
OXY);
>  sat_provider  =3D loc_network_has_flag (net, LOC_NETWORK_FLAG_SATELLITE_PR=
OVIDER);
>=20
>  if (AS_num) {
>    int r =3D loc_as_new (libloc_data.ctx, &as, AS_num);
>    if (r =3D=3D 0)
>       AS_name =3D loc_as_get_name (as);
>  }
>  printf ("ASN: %lu, AF: %d, country: %s, name: %s (%d,%d,%d)",
>          AS_num, family, country, AS_name, is_anycast,
>          is_anon_proxy, sat_provider);
>  if (as)
>     loc_as_unref (as);
>  loc_network_unref (net);
>=20
> -------------

Cool, this doesn=E2=80=99t look too bad.

First thing to know is that we use reference counting for all objects.

So every pointer that refers to an object has to decrement the counter when i=
t no longer needs it. The object will be automatically cleaned up after nothi=
ng needs it any more. Therefore the loc_*_unref() calls are always needed. If=
 you don=E2=80=99t call them, you will leak memory.

The next thing is that networks and AS are independent of each other. You can=
 have a network giving you an ASN, but there might be no AS object in the dat=
abase for it. That is because we might not have a name for that AS (ARIN does=
n=E2=80=99t publicly share them) and therefore we do not need to waste memory=
 on an empty object.

In your code, you are creating a new AS which is not what you want.

You want to call loc_database_get_as() with the AS number that you got from l=
oc_network_get_asn(). That function with then either return an AS or NULL.

loc_as_get_name will give you the name. So your code might look like this:

struct loc_network* network =3D loc_database_lookup_from_string(db, "81.3.27.=
38");
if (network) {
	uint32_t asn =3D loc_network_get_asn(network);

	if (asn) {
		struct loc_as* a =3D loc_database_get_as(db, asn);
		if (a) {
			name =3D loc_as_get_name(a);
		}
	}

	loc_network_unref(network);
}

// Copy the name to somewhere
...

// Cleanup
if (a)
	loc_as_unref(a);


> Everything except 'AS_name' is okay and prints sensible values
> (I've checked all ret-vals). Since there isn't any 'loc_get_asn_by_number()'
> function, what should I do to get a non-NULL 'AS_name' from 'AS_num'?
>=20
> PS. do I really need this 'loc_network_unref()'?
>  Or does 'loc_database_lookup()' handle it?

I hope this answers your questions :)

Best,
-Michael

--===============2089235709950895497==--