BUG: Series.unique segfaults on invalid unicode

[marco-neumann-by]

Not present anymore, might be fixed by accident, but no could not find a PR that did that.

Code Sample, a copy-pastable example

import pandas as pd

ser = pd.Series(["\ud83d"])
ser.unique()

Problem description

UnicodeEncodeError: 'utf-8' codec can't encode character '\ud83d' in position 0: surrogates not allowed
Exception ignored in: 'pandas._libs.tslibs.util.get_c_string_buf_and_size'                                             
UnicodeEncodeError: 'utf-8' codec can't encode character '\ud83d' in position 0: surrogates not allowed
Segmentation fault (core dumped)             

Expected Output

Not crashing.

Output of pd.show_versions()

INSTALLED VERSIONS

commit : None
python : 3.6.6.final.0
python-bits : 64
OS : Linux
OS-release : 5.4.0-33-generic
machine : x86_64
processor :
byteorder : little
LC_ALL : en_US.UTF-8
LANG : en_US.UTF-8
LOCALE : en_US.UTF-8

pandas : 1.0.4
numpy : 1.18.1
pytz : 2019.3
dateutil : 2.8.1
pip : 19.2.3
setuptools : 41.2.0
Cython : None
pytest : 5.4.1
hypothesis : None
sphinx : 3.0.3
blosc : None
feather : None
xlsxwriter : None
lxml.etree : None
html5lib : None
pymysql : None
psycopg2 : None
jinja2 : 2.11.2
IPython : 7.15.0
pandas_datareader: None
bs4 : None
bottleneck : None
fastparquet : None
gcsfs : None
lxml.etree : None
matplotlib : None
numexpr : None
odfpy : None
openpyxl : None
pandas_gbq : None
pyarrow : 0.15.1
pytables : None
pytest : 5.4.1
pyxlsb : None
s3fs : None
scipy : None
sqlalchemy : None
tables : None
tabulate : None
xarray : None
xlrd : None
xlwt : None
xlsxwriter : None
numba : None

[phofl]

Hey,

thanks for your report. This works for me on master. Output:

['\ud83d']

Which commit did you test on? I pulled before executing this.

[jorisvandenbossche]

This is indeed fixed on master. I suppose by #32701 and #33235

[marco-neumann-by]

Oh sorry, it seems I wasn't clear enough ("Not present anymore, might be fixed by accident, but no could not find a PR that did that."). I've meant that it currently is fixed on master, but I wasn't sure if it was fixed by some PR just "by accident" (i.e. some refactoring / bugfix that was meant to address something else) or if it is even present on master (SEGFAULTs can be somewhat tricky due to their dependency on memory layout and certain runtime aspects).

[jorisvandenbossche]

Ah, I read over your note (BTW, feel free to edit the top of the issue template, as I did now, it's mostly to remind people to do that when they are writing the issue, but you don't necessarily need to keep it once written. That can make your issue easier to read)

But indeed, I suppose it was kind of by accident (it was the same unicode string, but causing an error in another context, not a segfault), so at least we want to add tests for this case.

[arw2019]

@jorisvandenbossche @marco-neumann-jdas @phofl I'm happy to do a PR with these tests if nobody is working on this already!