Looks like this class hasn't really been changed since it was moved to
git, so I figured I'd fire out the question here to see if someone can
give me a hand.

I'm looking at https://bugzilla.redhat.com/show_bug.cgi?id=485085 . In
short, I'm seeing an error changing a base channel through the SSM when
you're changing from the default channel to the default channel (I know,
it's a weird case).

In particular BaseSubscribeAction line 166 is what's confusing me.

If I'm reading this right, we get here if you're changing to the default
base channel (it will be ID -1, which gets us into the condition at
164).

What I don't get is why we're loading a DistChannelMap for the old
channel. When my old channel is the default base, that map is coming
back as non-null. That's causing me to get to line 169, but ends up with
a null pointer exception at 172.

I could put a null check there, but I'm hesitant to until I understand
why we're looking to the map for the old channel to find the new
channel.

Any advice is appreciated.

-- 
Jason Dobies ([email protected])
RHN Satellite & Spacewalk
RHCE# 805008743336126
Freenode: jdob @ #spacewalk, #spacewalk-devel

_______________________________________________
Spacewalk-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/spacewalk-devel

Reply via email to