fun-dns proxy gives incorrect result

[CodeTrol]

Hi! I like fun-dns and I would like to use it as replacement to BIND9 for my home network. Why? Because BIND's recursion feature stopped working unexpectedly (even re-installation doesn't solve it), so I use fun-dns. I set up conf.json and serv.ini. However, when fun-dns uses proxy records come different from original. Let me show you by example:
nslookup google.com

# original (ISP's dns response)#
Non-authoritative answer:
Name:   google.com
Address: 173.194.39.129
Name:   google.com
Address: 173.194.39.130
Name:   google.com
Address: 173.194.39.131
Name:   google.com
Address: 173.194.39.132
Name:   google.com
Address: 173.194.39.133
Name:   google.com
Address: 173.194.39.134
Name:   google.com
Address: 173.194.39.135
Name:   google.com
Address: 173.194.39.136
Name:   google.com
Address: 173.194.39.137
Name:   google.com
Address: 173.194.39.142
Name:   google.com
Address: 173.194.39.128
# fun-dns response (using proxy) #
Name:   google.com
Address: 173.39.128.128
Name:   google.com
Address: 173.39.129.129
Name:   google.com
Address: 173.39.130.130
Name:   google.com
Address: 173.39.131.131
Name:   google.com
Address: 173.39.132.132
Name:   google.com
Address: 173.39.133.133
Name:   google.com
Address: 173.39.134.134
Name:   google.com
Address: 173.39.135.135
Name:   google.com
Address: 173.39.136.136
Name:   google.com
Address: 173.39.137.137
Name:   google.com
Address: 173.39.142.142

As you can see, numbers (a,b,c,d) in fun-dns shown like (a,c,d,d)

[CodeTrol]

I found a bug in /node_modules/ndns.js at function DNSParser.prototype.parseA (line 1366):

DNSParser.prototype.parseA = function () {
    if (this.parseStart + 4 > this.parseEnd)
    throw new Error();
    this.parseStart += 4;
    return [this.buf[this.parseStart-4],
        this.buf[this.parseStart-2],
        this.buf[this.parseStart-1],
        this.buf[this.parseStart-1]].join('.');
};

I changed to:

DNSParser.prototype.parseA = function () {
    if (this.parseStart + 4 > this.parseEnd)
    throw new Error();
    this.parseStart += 4;
    return [this.buf[this.parseStart-4],
        this.buf[this.parseStart-3],
        this.buf[this.parseStart-2],
        this.buf[this.parseStart-1]].join('.');
};

And problem was solved.

As I am not experienced github user, could you @badlee add this code?

Best wishes,
Victor Palyvoda.

[oshimin]

Thank's i fix in f48d757.