求高人指点一下这个存储过程哪地方有问题
每次搞存储过程，好多不会，尝试写了一个，运行不太正常，请教问题出在哪。

USE [db]

GO

/****** Object:  StoredProcedure [dbo].[XMUserAllot]    Script Date: 2013/8/29 23:29:56 ******/

SET ANSI_NULLS ON

GO

SET QUOTED_IDENTIFIER ON

GO

-- =============================================

-- Author:		<Author,,Name>

-- Create date: <Create Date,,>

-- Description:	<Description,,>

-- =============================================

ALTER PROCEDURE [dbo].[XMUserAllot]

	@UserName varchar(25)

AS

begin

	SET NOCOUNT ON;

	declare @UserId int,

	@ThisUser varchar(25),

	@AllotCount smallint,

	@RightCount smallint,

	@LeftCount smallint

	if(@UserName is null)

		declare XM_Cursor cursor forward_only static for select [username],[defaultshouquan] from ShouQuanUsers where defaultshouquan>0

	else

		declare XM_Cursor cursor forward_only static for select [username],[defaultshouquan] from ShouQuanUsers where defaultshouquan>0 and [username]=@UserName

	open XM_Cursor

	fetch next from XM_Cursor into @ThisUser,@AllotCount

	while(@@FETCH_STATUS=0)

	begin

		select @UserId=[Id] from [User] where UserName=@ThisUser

		if(@UserId is not null)

		begin

			select @RightCount=count(*) from XMUsers where userid=@UserId

			set @LeftCount= @AllotCount-@RightCount

			if(@LeftCount>0)

				update top(@LeftCount) XMUsers set userid=@UserId where userid is null

		end

		fetch next from XM_Cursor into @ThisUser,@AllotCount

		set @UserId=null

	end

	close XM_Cursor

	deallocate XM_Cursor

end

代码功能是按照表ShouQuanUsers记录每个用户应该分配到的商品数量，给用户分配礼品。现在的情况如果这个存储过程传递单一用户名进去，给这个用户分配的礼品是正常的，如果传递null进去，按道理来说是给所有用户进行礼品的分配，但是实际上并没有给所有用户进行分配，有的分了有的没分，分到的也没有分够设定的数量，这是怎么回事？我实在是搞不明白
------解决方案--------------------
--没看出逻辑上有什么问题。更新一个更新正常，更新所有中的那一个应该也正常。修改如下：

CREATE PROCEDURE [dbo].[XMUserAllot] @UserName VARCHAR(25)