Help, buged port scanner, can't find the bug :(

Question

Ok, i was already asking for help here and i got pretty good one. Now i did all the gui, but logic isn't working. When i press the scan everything just stucks...
This is the class that does scanning.
package portscanner;
import UI.ResultPanel;
import java.io.IOException;
import java.net.InetAddress;
import java.net.Socket;
import java.net.UnknownHostException;
public class Scanner {
    private Socket sock = null;
    private InetAddress hostip;
    private String hostname = "127.0.0.1";
    private ResultPanel rp = new ResultPanel();
    public void Scanner(){
    public void scan() {
    try {
            InetAddress addr = InetAddress.getByName(hostname);
            rp.resultArea.append("Scanning started...");
            rp.resultArea.repaint();
            for (int i = 1; i < 65536; i++) {
                try {
                    sock = new Socket(addr,i);
                    rp.resultArea.append("Port: " + i + "is open on " + addr);
                    rp.resultArea.repaint();
                catch (IOException ex) {
                finally {
                    try {
                        if (sock != null) sock.close();
                    catch (IOException ex) {}
            } //for
        } //try
        catch (UnknownHostException ex) {
            rp.resultArea.append("There has been an error. Aborting...");
            rp.repaint();
    public Socket getSock() {
        return sock;
    public void setSock(Socket sock) {
        this.sock = sock;
    public InetAddress getHostip() throws UnknownHostException {
        return hostip;
    public void setHostip(InetAddress hostip) {
        this.hostip = hostip;
    public ResultPanel getRp() {
        return rp;
    public void setRp(ResultPanel rp) {
        this.rp = rp;
Class that does the UI.
package UI;
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
public class UI extends JPanel{
    ResultPanel sp = new ResultPanel();
    private JLabel hostLabel = new JLabel("Host: ");
    private JTextField hostField = new JTextField(20);
    private JButton scanButton = new JButton("Scan");
    private JButton quitButton = new JButton("Quit");
    private JButton aboutButton = new JButton("About");
    public UI(ActionListener al){
        this.setLayout(new GridBagLayout());
        setButtons(al);
        fillPanel();
    private void fillPanel(){
    this.add(hostLabel, new GridBagConstraints(0, 0, 1, 1, 0, 0,
             GridBagConstraints.WEST,
             GridBagConstraints.NONE,
             new Insets(5, 0, 0, 5), 0, 0));
    this.add(hostField, new GridBagConstraints(1, 0, 20, 1, 0, 0,
             GridBagConstraints.CENTER,
             GridBagConstraints.HORIZONTAL,
             new Insets(5, 0, 0, 5), 0, 0));
    this.add(scanButton, new GridBagConstraints(0, 1, 1, 1, 0, 0,
             GridBagConstraints.WEST,
             GridBagConstraints.NONE,
             new Insets(5, 0, 0, 5), 0, 0));
    this.add(quitButton, new GridBagConstraints(1, 1, 1, 1, 0, 0,
             GridBagConstraints.CENTER,
             GridBagConstraints.NONE,
             new Insets(5, 0, 0, 5), 0, 0));
    this.add(aboutButton, new GridBagConstraints(2, 1, 1, 1, 0, 0,
             GridBagConstraints.EAST,
             GridBagConstraints.NONE,
             new Insets(5, 0, 0, 5), 0, 0));
    private void setButtons(ActionListener al){
    scanButton.setActionCommand("scan");
    scanButton.addActionListener(al);
    scanButton.setMnemonic('S');
    quitButton.setActionCommand("quit");
    quitButton.addActionListener(al);
    quitButton.setMnemonic('Q');
    aboutButton.setActionCommand("about");
    aboutButton.addActionListener(al);
    aboutButton.setMnemonic('A');
    public JLabel getHostLabel() {
        return hostLabel;
    public void setHostLabel(JLabel hostLabel) {
        this.hostLabel = hostLabel;
    public JTextField getHostField() {
        return hostField;
    public void setHostField(JTextField hostField) {
        this.hostField = hostField;
    public JButton getScanButton() {
        return scanButton;
    public void setScanButton(JButton scanButton) {
        this.scanButton = scanButton;
    public JButton getQuitButton() {
        return quitButton;
    public void setQuitButton(JButton quitButton) {
        this.quitButton = quitButton;
    public JButton getAboutButton() {
        return aboutButton;
    public void setAboutButton(JButton aboutButton) {
        this.aboutButton = aboutButton;
This is the class that does result panel.
package UI;
import java.awt.*;
import javax.swing.*;
public class ResultPanel extends JPanel {
    public JTextArea resultArea = new JTextArea(19, 20);
    public JScrollPane resultScroll = new JScrollPane(resultArea);
    public ResultPanel(){
    this.setBackground(Color.LIGHT_GRAY);
    this.add(resultScroll);
    public JTextArea getResultArea() {
        return resultArea;
    public void setResultArea(JTextArea resultArea) {
        this.resultArea = resultArea;
    public JScrollPane getResultScroll() {
        return resultScroll;
    public void setResultScroll(JScrollPane resultScroll) {
        this.resultScroll = resultScroll;
And the main...
package portscanner;
import UI.*;
import java.awt.*;
import java.awt.event.*;
import java.net.*;
import javax.swing.*;
public class Main extends JFrame implements ActionListener {
    JPanel mainPanel = new JPanel(new BorderLayout());
    Scanner sc = new Scanner();
    ResultPanel rp = new ResultPanel();
    UI window;
    Socket s;
    public Main() {
        window = new UI(this);
        window.setBackground(Color.LIGHT_GRAY);
        mainPanel.add(window, BorderLayout.CENTER);
        mainPanel.add(rp, BorderLayout.SOUTH);
        this.setContentPane(mainPanel);
    public static void main(String[] args) {
        JFrame.setDefaultLookAndFeelDecorated(false);
        JFrame mainWindow = new Main();
        mainWindow.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        mainWindow.setSize(300, 450);
        mainWindow.setLocationRelativeTo(null);
        mainWindow.setVisible(true);
    public void  actionPerformed(ActionEvent e){
        if(e.getActionCommand().equals("scan")){
        sc.scan();
        if(e.getActionCommand().equals("quit")){
Please help :S

Answer 1

Try this. I'm not experienced with Threads, this is just my stab at it.
public class Scanner extends Thread {
  public void run() {
    scan();
  private void scan() {
public class Main extends JFrame implements ActionListener {
  public void  actionPerformed(ActionEvent e){
        if(e.getActionCommand().equals("scan")){
          (new Scanner()).start();
        if(e.getActionCommand().equals("quit")){
}