<?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>0001249605</rptOwnerCik>
            <rptOwnerName>TUBB ALLEN R</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>Secretary</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>246</value>
        </transactionShares>
        <transactionPricePerShare>
          <value>0</value>
        </transactionPricePerShare>
        <transactionAcquiredDisposedCode>
          <value>A</value>
        </transactionAcquiredDisposedCode>
      </transactionAmounts>
      <postTransactionAmounts>
        <sharesOwnedFollowingTransaction>
          <value>246</value>
        </sharesOwnedFollowingTransaction>
      </postTransactionAmounts>
      <ownershipNature>
        <directOrIndirectOwnership>
          <value>D</value>
        </directOrIndirectOwnership>
      </ownershipNature>
    </nonDerivativeTransaction>
  </nonDerivativeTable>
  <derivativeTable>
    <derivativeHolding>
      <securityTitle>
        <value>Stock Option SWS Stock Option Plan</value>
      </securityTitle>
      <conversionOrExercisePrice>
        <value>16.6389</value>
      </conversionOrExercisePrice>
      <exerciseDate>
        <value>2000-10-10</value>
      </exerciseDate>
      <expirationDate>
        <value>2009-10-08</value>
      </expirationDate>
      <underlyingSecurity>
        <underlyingSecurityTitle>
          <value>Common Stock $.10 Par Value</value>
        </underlyingSecurityTitle>
        <underlyingSecurityShares>
          <value>1487</value>
        </underlyingSecurityShares>
      </underlyingSecurity>
      <postTransactionAmounts>
        <sharesOwnedFollowingTransaction>
          <value>1487</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>4917</value>
        </underlyingSecurityShares>
      </underlyingSecurity>
      <postTransactionAmounts>
        <sharesOwnedFollowingTransaction>
          <value>4917</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>2000</value>
        </underlyingSecurityShares>
      </underlyingSecurity>
      <postTransactionAmounts>
        <sharesOwnedFollowingTransaction>
          <value>2000</value>
        </sharesOwnedFollowingTransaction>
      </postTransactionAmounts>
      <ownershipNature>
        <directOrIndirectOwnership>
          <value>D</value>
        </directOrIndirectOwnership>
      </ownershipNature>
    </derivativeHolding>
  </derivativeTable>
  <footnotes />
  <remarks />
  <ownerSignature>
    <signatureName>Allen R. Tubb</signatureName>
    <signatureDate>2003-11-14</signatureDate>
  </ownerSignature>
</ownershipDocument>

