<?xml version="1.0"?>
<ownershipDocument>
    <schemaVersion>X0201</schemaVersion>

    <documentType>4</documentType>
  <periodOfReport>2003-11-12</periodOfReport>
    <issuer>
        <issuerCik>0000878520</issuerCik>
        <issuerName>SWS GROUP INC</issuerName>
        <issuerTradingSymbol>SWS</issuerTradingSymbol>
  </issuer>
  <reportingOwner>
        <reportingOwnerId>
            <rptOwnerCik>0001249603</rptOwnerCik>
            <rptOwnerName>LELAND DANIEL</rptOwnerName>
        </reportingOwnerId>
        <reportingOwnerAddress>
            <rptOwnerStreet1>1201 ELM STREET</rptOwnerStreet1>
            <rptOwnerStreet2>SUITE 3500</rptOwnerStreet2>
            <rptOwnerCity>DALLAS</rptOwnerCity>
            <rptOwnerState></rptOwnerState>
            <rptOwnerZipCode>75270</rptOwnerZipCode>
            <rptOwnerStateDescription></rptOwnerStateDescription>
        </reportingOwnerAddress>
            <reportingOwnerRelationship>
      <isOfficer>1</isOfficer>
      <officerTitle>Executive Vice President</officerTitle>
    </reportingOwnerRelationship>
  </reportingOwner>
  <nonDerivativeTable>
    <nonDerivativeTransaction>
      <securityTitle>
        <value>Common Stock $.10 Par Value</value>
      </securityTitle>
      <transactionDate>
        <value>2003-11-12</value>
      </transactionDate>
      <transactionCoding>
        <transactionFormType>4</transactionFormType>
        <transactionCode>A</transactionCode>
        <equitySwapInvolved>0</equitySwapInvolved>
      </transactionCoding>
      <transactionAmounts>
        <transactionShares>
          <value>615</value>
        </transactionShares>
        <transactionPricePerShare>
          <value>0</value>
        </transactionPricePerShare>
        <transactionAcquiredDisposedCode>
          <value>A</value>
        </transactionAcquiredDisposedCode>
      </transactionAmounts>
      <postTransactionAmounts>
        <sharesOwnedFollowingTransaction>
          <value>60973</value>
        </sharesOwnedFollowingTransaction>
      </postTransactionAmounts>
      <ownershipNature>
        <directOrIndirectOwnership>
          <value>D</value>
        </directOrIndirectOwnership>
      </ownershipNature>
    </nonDerivativeTransaction>
    <nonDerivativeHolding>
      <securityTitle>
        <value>Common Stock $.10 Par Value</value>
      </securityTitle>
      <postTransactionAmounts>
        <sharesOwnedFollowingTransaction>
          <value>4768</value>
        </sharesOwnedFollowingTransaction>
      </postTransactionAmounts>
      <ownershipNature>
        <directOrIndirectOwnership>
          <value>I</value>
        </directOrIndirectOwnership>
        <natureOfOwnership>
          <value>401 (k)</value>
        </natureOfOwnership>
      </ownershipNature>
    </nonDerivativeHolding>
    <nonDerivativeHolding>
      <securityTitle>
        <value>Common Stock $.10 Par Value</value>
      </securityTitle>
      <postTransactionAmounts>
        <sharesOwnedFollowingTransaction>
          <value>3450</value>
        </sharesOwnedFollowingTransaction>
      </postTransactionAmounts>
      <ownershipNature>
        <directOrIndirectOwnership>
          <value>I</value>
        </directOrIndirectOwnership>
        <natureOfOwnership>
          <value>Trust</value>
        </natureOfOwnership>
      </ownershipNature>
    </nonDerivativeHolding>
  </nonDerivativeTable>
  <derivativeTable>
    <derivativeHolding>
      <securityTitle>
        <value>Stock Option SWS Stock Option Plan</value>
      </securityTitle>
      <conversionOrExercisePrice>
        <value>13.7511</value>
      </conversionOrExercisePrice>
      <exerciseDate>
        <value>1998-08-27</value>
      </exerciseDate>
      <expirationDate>
        <value>2007-08-25</value>
      </expirationDate>
      <underlyingSecurity>
        <underlyingSecurityTitle>
          <value>Common Stock $.10 Par Value</value>
        </underlyingSecurityTitle>
        <underlyingSecurityShares>
          <value>7728</value>
        </underlyingSecurityShares>
      </underlyingSecurity>
      <postTransactionAmounts>
        <sharesOwnedFollowingTransaction>
          <value>7728</value>
        </sharesOwnedFollowingTransaction>
      </postTransactionAmounts>
      <ownershipNature>
        <directOrIndirectOwnership>
          <value>D</value>
        </directOrIndirectOwnership>
      </ownershipNature>
    </derivativeHolding>
    <derivativeHolding>
      <securityTitle>
        <value>Stock Option SWS Stock Option Plan</value>
      </securityTitle>
      <conversionOrExercisePrice>
        <value>12.4142</value>
      </conversionOrExercisePrice>
      <exerciseDate>
        <value>1999-08-24</value>
      </exerciseDate>
      <expirationDate>
        <value>2008-08-21</value>
      </expirationDate>
      <underlyingSecurity>
        <underlyingSecurityTitle>
          <value>Common Stock $.10 Par Value</value>
        </underlyingSecurityTitle>
        <underlyingSecurityShares>
          <value>7362</value>
        </underlyingSecurityShares>
      </underlyingSecurity>
      <postTransactionAmounts>
        <sharesOwnedFollowingTransaction>
          <value>7362</value>
        </sharesOwnedFollowingTransaction>
      </postTransactionAmounts>
      <ownershipNature>
        <directOrIndirectOwnership>
          <value>D</value>
        </directOrIndirectOwnership>
      </ownershipNature>
    </derivativeHolding>
    <derivativeHolding>
      <securityTitle>
        <value>Stock Option SWS Stock Option Plan</value>
      </securityTitle>
      <conversionOrExercisePrice>
        <value>26.0508</value>
      </conversionOrExercisePrice>
      <exerciseDate>
        <value>2000-08-23</value>
      </exerciseDate>
      <expirationDate>
        <value>2009-08-21</value>
      </expirationDate>
      <underlyingSecurity>
        <underlyingSecurityTitle>
          <value>Common Stock $.10 Par Value</value>
        </underlyingSecurityTitle>
        <underlyingSecurityShares>
          <value>7437</value>
        </underlyingSecurityShares>
      </underlyingSecurity>
      <postTransactionAmounts>
        <sharesOwnedFollowingTransaction>
          <value>7437</value>
        </sharesOwnedFollowingTransaction>
      </postTransactionAmounts>
      <ownershipNature>
        <directOrIndirectOwnership>
          <value>D</value>
        </directOrIndirectOwnership>
      </ownershipNature>
    </derivativeHolding>
    <derivativeHolding>
      <securityTitle>
        <value>Stock Option SWS Stock Option Plan</value>
      </securityTitle>
      <conversionOrExercisePrice>
        <value>23.3869</value>
      </conversionOrExercisePrice>
      <exerciseDate>
        <value>2001-08-24</value>
      </exerciseDate>
      <expirationDate>
        <value>2010-08-22</value>
      </expirationDate>
      <underlyingSecurity>
        <underlyingSecurityTitle>
          <value>Common Stock $.10 Par Value</value>
        </underlyingSecurityTitle>
        <underlyingSecurityShares>
          <value>6761</value>
        </underlyingSecurityShares>
      </underlyingSecurity>
      <postTransactionAmounts>
        <sharesOwnedFollowingTransaction>
          <value>6761</value>
        </sharesOwnedFollowingTransaction>
      </postTransactionAmounts>
      <ownershipNature>
        <directOrIndirectOwnership>
          <value>D</value>
        </directOrIndirectOwnership>
      </ownershipNature>
    </derivativeHolding>
    <derivativeHolding>
      <securityTitle>
        <value>Stock Option SWS Stock Option Plan</value>
      </securityTitle>
      <conversionOrExercisePrice>
        <value>15.4475</value>
      </conversionOrExercisePrice>
      <exerciseDate>
        <value>2002-09-10</value>
      </exerciseDate>
      <expirationDate>
        <value>2011-09-08</value>
      </expirationDate>
      <underlyingSecurity>
        <underlyingSecurityTitle>
          <value>Common Stock $.10 Par Value</value>
        </underlyingSecurityTitle>
        <underlyingSecurityShares>
          <value>11063</value>
        </underlyingSecurityShares>
      </underlyingSecurity>
      <postTransactionAmounts>
        <sharesOwnedFollowingTransaction>
          <value>11063</value>
        </sharesOwnedFollowingTransaction>
      </postTransactionAmounts>
      <ownershipNature>
        <directOrIndirectOwnership>
          <value>D</value>
        </directOrIndirectOwnership>
      </ownershipNature>
    </derivativeHolding>
    <derivativeHolding>
      <securityTitle>
        <value>Stock Option SWS Stock Option Plan</value>
      </securityTitle>
      <conversionOrExercisePrice>
        <value>13.42</value>
      </conversionOrExercisePrice>
      <exerciseDate>
        <value>2003-08-22</value>
      </exerciseDate>
      <expirationDate>
        <value>2012-08-19</value>
      </expirationDate>
      <underlyingSecurity>
        <underlyingSecurityTitle>
          <value>Common Stock $.10 Par Value</value>
        </underlyingSecurityTitle>
        <underlyingSecurityShares>
          <value>18000</value>
        </underlyingSecurityShares>
      </underlyingSecurity>
      <postTransactionAmounts>
        <sharesOwnedFollowingTransaction>
          <value>18000</value>
        </sharesOwnedFollowingTransaction>
      </postTransactionAmounts>
      <ownershipNature>
        <directOrIndirectOwnership>
          <value>D</value>
        </directOrIndirectOwnership>
      </ownershipNature>
    </derivativeHolding>
  </derivativeTable>
  <footnotes />
  <remarks />
  <ownerSignature>
    <signatureName>Allen R. Tubb, Power of Attorney</signatureName>
    <signatureDate>2003-11-14</signatureDate>
  </ownerSignature>
</ownershipDocument>

