< Previous by Date Date Index Next by Date >
  Thread Index Next in Thread >

[reSIProcate] ares_mkquery.c doesn't compile after last commit

Hi all,

I aligned my reSIProcate repository to origin master and I noticed that ares doesn’t build anymore under Windows.

Issue has been introduced in commit 920273631b699bcaf9ea3c2c448eb9af5a924b70 (26/06/2017) due to missing MAXCDNAME definition under Windows (I tested ares_14_0.vcxproj using Visual Studio 2017 Enterprise).

1>------ Build started: Project: ares, Configuration: Debug Win32 ------


1>ares_mkquery.c(124): warning C4267: '=': conversion from 'size_t' to 'unsigned char', possible loss of data

1>ares_mkquery.c(153): error C2065: 'MAXCDNAME': undeclared identifier

1>ares_mkquery.c(153): warning C4018: '>': signed/unsigned mismatch

1>Done building project "ares_14_0.vcxproj" -- FAILED.


I don’t know how we want to solve the issue.

Maybe we could change ares_mkquery.c adding definition for MAXCDNAME (as in nameser.h).

#ifndef WIN32

#include <netinet/in.h>

#ifndef __CYGWIN__

#  include <arpa/nameser.h>



#define MAXCDNAME   255     /* maximum compressed domain name */



Or we could change ares.h  adding MAXCDNAME (after MAXHOSTNAMELEN?).

#define PACKETSZ 512







#define MAXCDNAME   255     /* maximum compressed domain name */



I prefer the change in ares.h. Other comments / suggestions?


Best regards,