We currently have this:
/* TODO: Autogenerate this somehow */
r->dwSerial = 110;
which seems to be wrong.
In the rpc server we have a
dnsserver_update_soa() function which increments the searial number
on the SOA record.
[MS-DNSP] says "dwSerial: This MUST be set to 0x00000000 when sent by the client or server, and ignored on receipt by the server or client" and "dwFlags, dwSerial, dwTimeStamp, and dwReserved are set to zero". Except for SOA records.
Could zero be the correct autogeneration? Or is there a chance that an SOA could come through these paths and we need to handle that case?
110 is also used with despairing comments in py_dsdb_dns_replace_by_dn() and py_dsdb_dns_replace().