Comparison

net.server_event: Don't emit empty packets, check for errors first. (fixes #287)

comparison

equal deleted inserted replaced

-:b27ba2c83dd4
+:2f8f388de516
 						interface.eventreadtimeout = nil
 					end
 				end
 				local buffer, err, part = interface.conn:receive( interface._pattern )  -- receive buffer with "pattern"
 				--vdebug( "read data:", tostring(buffer), "error:", tostring(err), "part:", tostring(part) )
-				buffer = buffer or part or ""
+				buffer = buffer or part
-				local len = string_len( buffer )
+				if buffer and #buffer > cfg.MAX_READ_LENGTH then  -- check buffer length
-				if len > cfg.MAX_READ_LENGTH then  -- check buffer length
 					interface.fatalerror = "receive buffer exceeded"
 					debug( "fatal error:", interface.fatalerror )
 					interface:_close()
 					interface.eventread = nil
 					return -1
 				end
-				interface.onincoming( interface, buffer, err )  -- send new data to listener
 				if err and ( err ~= "timeout" and err ~= "wantread" ) then
 					if "wantwrite" == err then -- need to read on write event
 						if not interface.eventwrite then  -- register new write event if needed
 							interface.eventwrite = addevent( base, interface.conn, EV_WRITE, interface.writecallback, cfg.WRITE_TIMEOUT )
 						end
 						debug( "connection failed in read event:", interface.fatalerror )
 						interface:_close()
 						interface.eventread = nil
 						return -1
 					end
+				else
+					interface.onincoming( interface, buffer, err )  -- send new data to listener
 				end
 				if interface.noreading then
 					interface.eventread = nil;
 					return -1;
 				end