Use config tftpBindAddress instead of dhcpGateway for siaddr and sname

dpasupathi · May 10, 2016 · 82eea3b · 82eea3b
1 parent ef967b9
commit 82eea3b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/packet.js b/lib/packet.js
@@ -45,9 +45,9 @@ function PacketFactory(protocol, configuration, _) {
             packet.fname = bootFileName;
 
             // Necessary, at least on vbox
-            packet.siaddr = configuration.get('dhcpGateway', '10.1.1.1');
+            packet.siaddr = configuration.get('tftpBindAddress', '10.1.1.1');
             // Not necessary, at least on vbox, but perhaps other clients will require these fields?
-            packet.sname = configuration.get('dhcpGateway', '10.1.1.1');
+            packet.sname = configuration.get('tftpBindAddress', '10.1.1.1');
 
             //EFI PXE listen on a different port => tell the server
             if ((packet.options.userClass === undefined) &&

diff --git a/spec/lib/packet-spec.js b/spec/lib/packet-spec.js
@@ -73,7 +73,7 @@ describe("Packet", function() {
 
         it("should create a proper ACK packet for proxyDHCP", function() {
             var testbootfile = 'testbootfile';
-            configuration.get.withArgs('dhcpGateway').returns('10.1.1.1');
+            configuration.get.withArgs('tftpBindAddress').returns('10.1.1.1');
             configuration.get.withArgs('broadcastaddr').returns('10.1.1.255');
 
             packetUtil.createProxyDhcpAck(testPacket, testbootfile);