Ma è sicuramente molto "dubbia". Quel sentence, per quanto si può intuire dal codice, è sicuramente un campo static. E già qui è un male ...
Ma non è tutto: UDPServer è sicuramente un Runnable (lo passi al costruttore di Thread). Questo vuol dire che dentro UDPServer "girerà" del codice in un thread differente da quello che esegue il codice che hai postato. E vedendo l'uso diretto di quel campo static, vuol anche dire che, molto probabilmente, non hai nemmeno considerato le questioni relative alla sincronizzazione e "visibilità" delle modifiche tra i thread.