Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2e514606 authored by Ken Chen's avatar Ken Chen Committed by Automerger Merge Worker
Browse files

Fix -Wnullable-to-nonnull-conversion errors am: 4730f0d7 

Original change: https://android-review.googlesource.com/c/platform/packages/modules/DnsResolver/+/2582958



Change-Id: Ie0fc71d0495b5a261a3b58881a11574af5507c81
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 2d0750b2 4730f0d7

gethnamaddr.cpp

+6 −3
Original line number Original line Diff line number Diff line
@@ -279,9 +279,11 @@ static struct hostent* getanswer(const querybuf* _Nonnull answer, int anslen, co 
                }

                }

                break;

                break;

            case T_A:

            case T_A:

            case T_AAAA:

            case T_AAAA: {

                if (strcasecmp(hent->h_name, bp) != 0) {

                if (hent->h_name == NULL) goto no_recovery;

                    LOG(DEBUG) << __func__ << ": asked for \"" << hent->h_name << "\", got \"" << bp

                const char* h_name = hent->h_name;

                if (strcasecmp(h_name, bp) != 0) {

                    LOG(DEBUG) << __func__ << ": asked for \"" << h_name << "\", got \"" << bp

                               << "\"";

                               << "\"";

                    cp += n;

                    cp += n;

                    continue; /* XXX - had_error++ ? */

                    continue; /* XXX - had_error++ ? */
@@ -325,6 +327,7 @@ static struct hostent* getanswer(const querybuf* _Nonnull answer, int anslen, co 
                cp += n;

                cp += n;

                if (cp != erdata) goto no_recovery;

                if (cp != erdata) goto no_recovery;

                break;

                break;

            }

            default:

            default:

                abort();

                abort();

        }

        }

sethostent.cpp

+6 −6
Original line number Original line Diff line number Diff line
@@ -101,7 +101,12 @@ int _hf_gethtbyname2(const char* name, int af, getnamaddr* info) { 
            break;

            break;

        }

        }





        if (strcasecmp(hp->h_name, name) != 0) {

        if (hp->h_name == nullptr) {

            free(buf);

            return EAI_FAIL;

        }

        const char* h_name = hp->h_name;

        if (strcasecmp(h_name, name) != 0) {

            char** cp;

            char** cp;

            for (cp = hp->h_aliases; *cp != NULL; cp++)

            for (cp = hp->h_aliases; *cp != NULL; cp++)

                if (strcasecmp(*cp, name) == 0) break;

                if (strcasecmp(*cp, name) == 0) break;
@@ -113,11 +118,6 @@ int _hf_gethtbyname2(const char* name, int af, getnamaddr* info) { 
            hent.h_addrtype = hp->h_addrtype;

            hent.h_addrtype = hp->h_addrtype;

            hent.h_length = hp->h_length;

            hent.h_length = hp->h_length;





            if (hp->h_name == nullptr) {

                free(buf);

                return EAI_FAIL;

            }

            const char* h_name = hp->h_name;

            HENT_SCOPY(hent.h_name, h_name, ptr, len);

            HENT_SCOPY(hent.h_name, h_name, ptr, len);

            for (anum = 0; hp->h_aliases[anum]; anum++) {

            for (anum = 0; hp->h_aliases[anum]; anum++) {

                if (anum >= MAXALIASES) goto nospc;

                if (anum >= MAXALIASES) goto nospc;