sql-server - 'Try/Catch' ブロック内の SQL Server 'Execute As'/Revert パターン

Question

SQL Server 2012 の Try/Catch ブロック内から Execute As/Revert を使用するときに、「最適な」パターンを使用していることを確認したいと思います。以下のコードは正しく動作するように「思われます」...セキュリティ上の懸念、「より良い」アプローチなどはありますか?

以下は私のコードです：

CREATE PROCEDURE [dbo].[TryItOut]
WITH EXECUTE as 'NoTable1Access' --does not have access (select) to Table1!
AS
Begin
  Declare @execErrorNumber int,
           @execErrorMessage nvarchar(2048),
           @xactState smallint

  Begin Try
    Execute as user='HasTable1Access'
    select *, 1/0 as [SimulateError] from [T1].[Table1]; -- This will be a Stored Procedure call, but a select statement was easier to demo...
    Revert --Revert on 'HasTable1Access'
  End Try
  Begin Catch;

        select @execErrorNumber = ERROR_NUMBER(),
               @execErrorMessage = ERROR_MESSAGE(),
               @xactState = XACT_STATE();

        Revert -- Revert on 'HasTable1Access' when in error...
        --Do some error processing in context of 'NoTable1Access'
   End Catch 

   select * from [T1].[Table1] --Should NOT have any access now and select should fail...
End